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

feat: Implement OR for PromQL #3024

Merged
merged 7 commits into from
Dec 28, 2023
Merged

Conversation

waynexia
Copy link
Member

I hereby agree to the terms of the GreptimeDB CLA

What's changed and what's your intention?

This PR implements OR operator for PromQL. See UnionDistinctOn's document for detailed explain.

Notice that there are a few unresolved problems in this patch. I'll file another one to resolve them since this PR reaches it size limit

Checklist

  • I have written the necessary rustdoc comments.
  • I have added the necessary unit tests and integration tests.
  • This PR does not require documentation updates.

Refer to a related PR or issue link (optional)

#1042

Signed-off-by: Ruihang Xia <[email protected]>
Signed-off-by: Ruihang Xia <[email protected]>
Signed-off-by: Ruihang Xia <[email protected]>
Signed-off-by: Ruihang Xia <[email protected]>
Signed-off-by: Ruihang Xia <[email protected]>
Signed-off-by: Ruihang Xia <[email protected]>
Copy link

codecov bot commented Dec 27, 2023

Codecov Report

Attention: 355 lines in your changes are missing coverage. Please review.

Comparison is base (feec4e2) 85.70% compared to head (9dc27af) 85.02%.
Report is 7 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3024      +/-   ##
===========================================
- Coverage    85.70%   85.02%   -0.69%     
===========================================
  Files          784      793       +9     
  Lines       126856   128346    +1490     
===========================================
+ Hits        108726   109129     +403     
- Misses       18130    19217    +1087     

Copy link
Contributor

@killme2008 killme2008 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job! A few questions and suggestions, please take a look.

src/promql/src/planner.rs Outdated Show resolved Hide resolved
src/promql/src/planner.rs Show resolved Hide resolved
@evenyag evenyag changed the base branch from develop to main December 28, 2023 03:50
@killme2008 killme2008 added this pull request to the merge queue Dec 28, 2023
Merged via the queue into GreptimeTeam:main with commit b58296d Dec 28, 2023
17 checks passed
@waynexia waynexia deleted the prom-or branch January 2, 2024 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs-not-required This change does not impact docs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants