Closed Bug 919607 Opened 12 years ago Closed 12 years ago

Cache aggregated values in telemetry.js for better performance

Categories

(Data Platform and Tools :: General, defect)

Product:
Data Platform and Tools
Component:
General
Type:
defect
Priority:
Not set
Severity:
minor

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: jonasfj, Unassigned)

References

Details

In `telemetry.js` we do a lot of data aggregation... As histograms grow in number of build dates, we might want to consider materializing some of the histograms perhaps HistogramEvolution.range() would be a decent place to do to this... Another solution would be to materialize on-the-fly, ie. whenever the auxiliary `function _aggregate(index, histogram)` is called. Note, this might very well be a "won't fix". Only time and more data will tell if current approach will give us client side performance issues.
Summary: Consider caching aggregated values in telemetry.js for better performance → Cache aggregated values in telemetry.js for better performance
This might be fixed in https://github.com/jonasfj/telemetry-dashboard/commit/3bb5abc820bc4a867f23daf6655e0f1add08a2a9 We might be able to do more to speed this up... Anyways, I should look more at this on Monday.
Okay, I think this is fixed. reopen if you disagree :)
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Product: Webtools → Data Platform and Tools
Component: Telemetry Dashboards (TMO) → General
You need to log in before you can comment on or make changes to this bug.