mozilla / missioncontrol

Real-time monitoring of Firefox release health
Mozilla Public License 2.0
21 stars 18 forks source link

Don't update graphs unnecessarily in detail view #172

Closed wlach closed 6 years ago

wlach commented 6 years ago

We were previously updating the measure/time graphs in the detail view every time any aspect of the global state changed, even when such a state did not effect the graphs at all (e.g. the user was just changing the time picker). This operation could be extremely slow when we're visualizing a large amount of data. Use react's shouldComponentUpdate functionality to avoid unnecessary updates.