From 1ae96cefad7c46abd7af35fca12d8c227102a4a3 Mon Sep 17 00:00:00 2001 From: Ben Cooke Date: Fri, 23 Feb 2024 13:11:25 -0500 Subject: [PATCH] [MM-56901] Exclude selectors that have less than 5 calls from telemetry (#26246) * exclude selectors from telemetry that have less than 5 calls --------- Co-authored-by: Mattermost Build --- .../src/actions/telemetry_actions.jsx | 19 ++++++++++--------- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git a/webapp/channels/src/actions/telemetry_actions.jsx b/webapp/channels/src/actions/telemetry_actions.jsx index 625bdf1275..5b173a41d4 100644 --- a/webapp/channels/src/actions/telemetry_actions.jsx +++ b/webapp/channels/src/actions/telemetry_actions.jsx @@ -211,18 +211,19 @@ export function trackSelectorMetrics() { } const selectors = getSortedTrackedSelectors(); + const filteredSelectors = selectors.filter((selector) => selector.calls > 5); trackEvent('performance', 'least_effective_selectors', { after: 'one_minute', - first: selectors[0]?.name || '', - first_effectiveness: selectors[0]?.effectiveness, - first_recomputations: selectors[0]?.recomputations, - second: selectors[1]?.name || '', - second_effectiveness: selectors[1]?.effectiveness, - second_recomputations: selectors[1]?.recomputations, - third: selectors[2]?.name || '', - third_effectiveness: selectors[2]?.effectiveness, - third_recomputations: selectors[2]?.recomputations, + first: filteredSelectors[0]?.name || '', + first_effectiveness: filteredSelectors[0]?.effectiveness, + first_recomputations: filteredSelectors[0]?.recomputations, + second: filteredSelectors[1]?.name || '', + second_effectiveness: filteredSelectors[1]?.effectiveness, + second_recomputations: filteredSelectors[1]?.recomputations, + third: filteredSelectors[2]?.name || '', + third_effectiveness: filteredSelectors[2]?.effectiveness, + third_recomputations: filteredSelectors[2]?.recomputations, }); }, 60000); }