-
Notifications
You must be signed in to change notification settings - Fork 338
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
fix: adjust fill behavior of range query #3489
Conversation
34eca21
to
3eecb3e
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #3489 +/- ##
==========================================
- Coverage 85.40% 85.03% -0.37%
==========================================
Files 897 898 +1
Lines 148106 148658 +552
==========================================
- Hits 126495 126417 -78
- Misses 21611 22241 +630 |
@tisonkun Is this behavior you expected? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I hereby agree to the terms of the GreptimeDB CLA.
What's changed and what's your intention?
Adjust fill behavior on range query
fill
, the null result calculate by range expr, or the missing datapoint will not be fill.NULL
, the missing datapoint will be fill.LINEAR/PREV/Const
, all null value and missing datapoint will be fill.Case like
Checklist