Skip to content

promql: convert QueryOpts to interface#12516

Merged
roidelapluie merged 1 commit intoprometheus:mainfrom
vinted:convert_queryopts_to_interface
Jul 4, 2023
Merged

promql: convert QueryOpts to interface#12516
roidelapluie merged 1 commit intoprometheus:mainfrom
vinted:convert_queryopts_to_interface

Conversation

@GiedriusS
Copy link
Contributor

Convert QueryOpts to an interface so that downstream projects like https://github.com/thanos-community/promql-engine could extend the query options with engine specific options that are not in the original engine.

Will be used to enable query analysis per-query.

@GiedriusS GiedriusS requested a review from roidelapluie as a code owner July 3, 2023 12:57
Convert QueryOpts to an interface so that downstream projects like
https://github.com/thanos-community/promql-engine could extend the query
options with engine specific options that are not in the original
engine.

Will be used to enable query analysis per-query.

Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com>
@GiedriusS
Copy link
Contributor Author

CI failure seems unrelated to my changes.

@roidelapluie roidelapluie self-assigned this Jul 4, 2023
@roidelapluie roidelapluie merged commit 0186ec7 into prometheus:main Jul 4, 2023
@roidelapluie
Copy link
Member

Thanks!

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.

2 participants