1893782 - "Use Replicates" on treeherder spends a ton of time around icu

Status: NEW

(Tree Management :: Perfherder, task)

Description

[Mayank Bansal]

Go to https://treeherder.mozilla.org/perfherder/comparesubtest?originalProject=try&newProject=try&newRevision=71e6ec76466ac6b2c0c2172af5ef8101e79cb3b7&originalSignature=4586009&newSignature=4586009&framework=13&application=firefox&originalRevision=494499dbf98a5df49e9c69576e344df25059ee8e&page=1&replicates=1

Let the page load
Click on "Show only important changes". Let the page load
Unclick on "Show only important changes". Let the page load

Nightly: https://share.firefox.dev/3UzHGAa
Chrome: https://share.firefox.dev/49Vs12w
ER: Maybe we can be better here?

Not sure if this is something that needs to be fixed in treeherder scripts (lodash?) or JS or ICU. Feel free to reclassify.
Edit: Chrome seems to take almost the same time as Nightly. So I am guessing the "fix" needs to be in treeherder.

Comment 1

[André Bargull [:anba]]

The formatNumber function in perfherder/perf-helpers/helpers.js creates a new Intl.NumberFormat instance on each call. Creating the Intl.NumberFormat object only once should make this much faster.

Comment 2

[Greg Mierzwinski [:sparky]]

Thanks for filing a bug for this! We'll look into it.