Skip to content

Conversation

@k-fish
Copy link
Member

@k-fish k-fish commented Oct 16, 2025

Summary

Trace metrics has a blocker of having the available group-by and autocomplete keys being limited to the metric name selected instead of every attribute key ever ingested. Due to how the trace item filter rpc (and eap tables) are currently setup, this is only currently possible with an 'AND' on the key presence, so this is a workaround adding two temporary meta attributes that will let us experiment with this behaviour before open beta.

more details here

@k-fish k-fish requested a review from a team as a code owner October 16, 2025 15:51
@k-fish k-fish force-pushed the feat/tracemetrics/add-cooccuring-attribute-names branch from 0a11ee5 to 705b19a Compare October 16, 2025 15:56
Trace metrics has a blocker of having the available group-by and
autocomplete keys being limited to the metric name selected instead of
every attribute key ever ingested. Due to
how the trace item filter rpc (and eap tables) are currently setup, this
is only currently possible with an 'AND' on the key presence, so this is
a workaround adding two temporary meta attributes that will let us
experiment with this behaviour before open beta.
@k-fish k-fish force-pushed the feat/tracemetrics/add-cooccuring-attribute-names branch from 705b19a to ebed931 Compare October 16, 2025 16:01
@k-fish k-fish requested a review from Dav1dde October 16, 2025 18:47
@k-fish k-fish enabled auto-merge October 16, 2025 19:21
@k-fish k-fish added this pull request to the merge queue Oct 17, 2025
@Dav1dde Dav1dde removed this pull request from the merge queue due to a manual request Oct 17, 2025
@Dav1dde Dav1dde enabled auto-merge October 17, 2025 06:25
@Dav1dde Dav1dde added this pull request to the merge queue Oct 17, 2025
Merged via the queue into master with commit 4d083ff Oct 17, 2025
28 checks passed
@Dav1dde Dav1dde deleted the feat/tracemetrics/add-cooccuring-attribute-names branch October 17, 2025 06:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants