Delete expired Necko histograms & accumulation code

NEW
Unassigned

Status

()

P3
normal
4 years ago
8 months ago

People

(Reporter: vladan, Unassigned)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(firefox42 affected)

Details

(Whiteboard: [necko-triaged])

(Reporter)

Description

4 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)

Comment 1

4 years ago
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)

Updated

4 years ago
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)
(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)
Component: Telemetry → Networking
Product: Toolkit → Core

Updated

8 months ago
Priority: -- → P3

Updated

8 months ago
Whiteboard: [necko-triaged]
You need to log in before you can comment on or make changes to this bug.