mozilla / glam

Mozilla's primary interactive dashboard for examining the distribution of telemetry values.
https://glam.telemetry.mozilla.org
Mozilla Public License 2.0
20 stars 23 forks source link

Don't include Glean metrics on landing page to avoid api calls for metadata #2830

Closed edugfilho closed 1 week ago

edugfilho commented 1 month ago

This PR fixes a bug in which no probe is shown on the landing page. The root cause was actually a confusion related to UNION ALL with SELECT * where the columns matched types but were not the "same" columns.

After iterating on the solution for a while I also found out that if we want to include Glean metrics in this mix then GLAM would need to make an api call to Glean Dictionary for every metric included in the resultset, so I decided to remove Glean metrics from this feature for the time being, especially because it's not a very important feature of GLAM.