-
Notifications
You must be signed in to change notification settings - Fork 968
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
[To rel/1.0] [IOTDB-4940] Optimize query fetch data partition process while containg unclosed time filter #8230
Conversation
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.
PTAL~
// it will be set to true in query when there exist filter like: time <= XXX | ||
// (-oo, timePartitionSlotList.get(0)] | ||
private boolean needLeftAll = false; | ||
|
||
// it will be set to true query when there exist filter like: time >= XXX | ||
// [timePartitionSlotList.get(timePartitionSlotList.size() - 1), +oo) | ||
private boolean needRightAll = false; |
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.
The annotation shows that these two parameters indicate two close intervals. However in the implementation of SeriesPartitionTable, these two parameters indicate two open intervals.
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.
SonarCloud Quality Gate failed. |
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~
Optimize sqls like
where time < 10
where time > 20
where time < 10 or time>20