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: support bool operator with other computation #1844

Merged
merged 6 commits into from
Jun 29, 2023

Conversation

waynexia
Copy link
Member

I hereby agree to the terms of the GreptimeDB CLA

What's changed and what's your intention?

This patch:

  • supports query like metric + (1 > bool 2)
  • supports ^(power) and atan2 operators
  • rewrites the instant manipulator, it should be exactly the same with Prom's

Checklist

  • I have written the necessary rustdoc comments.
  • I have added the necessary unit tests and integration tests.

Refer to a related PR or issue link (optional)

waynexia added 6 commits June 27, 2023 19:10
Signed-off-by: Ruihang Xia <waynestxia@gmail.com>
Signed-off-by: Ruihang Xia <waynestxia@gmail.com>
Signed-off-by: Ruihang Xia <waynestxia@gmail.com>
Signed-off-by: Ruihang Xia <waynestxia@gmail.com>
Signed-off-by: Ruihang Xia <waynestxia@gmail.com>
@codecov
Copy link

codecov bot commented Jun 28, 2023

Codecov Report

Merging #1844 (1291c14) into develop (b737a24) will decrease coverage by 0.32%.
The diff coverage is 75.35%.

@@             Coverage Diff             @@
##           develop    #1844      +/-   ##
===========================================
- Coverage    86.53%   86.21%   -0.32%     
===========================================
  Files          588      589       +1     
  Lines        95724    96063     +339     
===========================================
- Hits         82831    82824       -7     
- Misses       12893    13239     +346     

Copy link
Contributor

@evenyag evenyag left a comment

Choose a reason for hiding this comment

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

LGTM 😢 🔫

@waynexia waynexia enabled auto-merge (squash) June 29, 2023 04:03
@waynexia waynexia requested a review from killme2008 June 29, 2023 07:07
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.

LGTM

@waynexia waynexia merged commit 605776f into GreptimeTeam:develop Jun 29, 2023
@waynexia waynexia deleted the literal-bool branch June 29, 2023 12:10
paomian pushed a commit to paomian/greptimedb that referenced this pull request Oct 19, 2023
* add some cases

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>

* fix sqlness test

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>

* impl atan2 and power

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>

* fix instant manipulator

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>

* fix clippy

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>

---------

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>
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