Delete expired Necko histograms & accumulation code

NEW
Unassigned

Status

()

Toolkit
Telemetry
2 years ago
2 years ago

People

(Reporter: vladan, Unassigned)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(firefox42 affected)

Details

(Reporter)

Description

2 years ago
A lot of Necko histograms have expired and need to be removed from the codebase along with their accumulation code.

Full list of expired histograms:
https://groups.google.com/forum/#!topic/mozilla.dev.telemetry-alerts/Uw9svhVTrXk

There are about a dozen expired Necko histograms in the list above:

UDP_SOCKET_PARALLEL_CLOSE_COUNT
DISK_CACHE_
LOCALDOMSTORAGE_
SESSIONDOMSTORAGE_
Flags: needinfo?(jduell.mcbugs)
Honza/Michal: the majority of these probes are from the cache/localstorage.  Can you verify that we're fine removing them all?
Flags: needinfo?(jduell.mcbugs) → needinfo?(honzab.moz)
Flags: needinfo?(michal.novotny)
Michal should decide on these:
NETWORK_CACHE_FS_TYPE
NETWORK_CACHE_SIZE_FULL_FAT

On these I vote to remove:
CACHE_SERVICE_LOCK*
NETWORK_DISK_CACHE_*
DISK_CACHE_*
LOCALDOMSTORAGE_* since our localStorage impl is far best of all browsers and I don't want to invest time in more optimizations any time sooner.  we can introduce the probes again if needed ; I'll do the removal on these
SESSIONDOMSTORAGE_*
Flags: needinfo?(honzab.moz)
Blocks: 1160903
(In reply to Honza Bambas (not reviewing) (:mayhemer) from comment #2)
> Michal should decide on these:
> NETWORK_CACHE_FS_TYPE
> NETWORK_CACHE_SIZE_FULL_FAT

These can be removed. The amount of data we've collected is enough.
Flags: needinfo?(michal.novotny)
You need to log in before you can comment on or make changes to this bug.