Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SSE: DSNode to update result with names to make each value identifiable by labels (only Graphite and TestData) #74615

Merged
merged 1 commit into from
Sep 8, 2023

Conversation

yuri-tceretian
Copy link
Contributor

@yuri-tceretian yuri-tceretian commented Sep 8, 2023

Backport of #71246 to 9.4.x. Actually, a cherry-pick of commit fb4796e from 9.5.x to 9.4.x

…le by labels (only Graphite and TestData) (#73642)

* SSE: DSNode to update result with names to make each value identifiable by labels (only Graphite and TestData) (#71246)

* introduce a function checkIfSeriesNeedToBeFixed to scan all value fields in the response and provide a function that updates Series so they can be uniquely identifiable. Only Graphite and TestData are checked.

* update `DSNode.Execute` to run this function and provide it to WideToMany
* update WideToMany to run the fix function if it is not nil
@yuri-tceretian yuri-tceretian added add to changelog area/expressions Server Side Expressions (SSE) backport A backport PR labels Sep 8, 2023
@yuri-tceretian yuri-tceretian requested review from a team as code owners September 8, 2023 15:56
@yuri-tceretian yuri-tceretian requested review from wbrowne and oshirohugo and removed request for a team September 8, 2023 15:56
@grafana-delivery-bot grafana-delivery-bot bot added this to the 9.4.x milestone Sep 8, 2023
@yuri-tceretian yuri-tceretian self-assigned this Sep 8, 2023
@armandgrillet armandgrillet added the product-approved Pull requests that are approved by product/managers and are allowed to be backported label Sep 8, 2023
@yuri-tceretian yuri-tceretian merged commit 82ac243 into v9.4.x Sep 8, 2023
@yuri-tceretian yuri-tceretian deleted the backport-71246-9.4.x branch September 8, 2023 19:04
@zerok zerok modified the milestones: 9.4.x, 9.4.16 Sep 19, 2023
@zerok zerok modified the milestones: 9.4.16, 9.4.17 Oct 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
add to changelog area/backend area/expressions Server Side Expressions (SSE) backport A backport PR product-approved Pull requests that are approved by product/managers and are allowed to be backported
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants