Skip to content

Fix LIMIT/OFFSET push down in GROUP BY MONTH#11714

Merged
JackieTien97 merged 10 commits intoapache:masterfrom
ycycse:monthPushDown
Dec 18, 2023
Merged

Fix LIMIT/OFFSET push down in GROUP BY MONTH#11714
JackieTien97 merged 10 commits intoapache:masterfrom
ycycse:monthPushDown

Conversation

@ycycse
Copy link
Member

@ycycse ycycse commented Dec 13, 2023

This PR fix the following problems:

  1. Can't use limit/offset with GROUP BY MONTH
  2. Wrong way to query with GROUP BY MONTH.
  3. Can't use __endTime in GROUP BY TIME + ALIGN BY DEVICE

1. group by ([2016-7-30),1mo) results in 2017-7-31 due to 2017-2-28
2. group by ([2016-7-31),1mo) results in 2017-7-28 due to 2017-2-28
…it's only supported in Group by month without Align by device and in ascending order)
@ycycse ycycse changed the title Fix LIMIT/OFFSET in GROUP BY MONTH Fix LIMIT/OFFSET push down in GROUP BY MONTH Dec 13, 2023
# Conflicts:
#	iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/execution/aggregation/timerangeiterator/AggrWindowIterator.java
#	iotdb-core/datanode/src/main/java/org/apache/iotdb/db/utils/DateTimeUtils.java
#	iotdb-core/tsfile/src/main/java/org/apache/iotdb/tsfile/read/filter/operator/GroupByMonthFilter.java
#	iotdb-core/tsfile/src/main/java/org/apache/iotdb/tsfile/utils/TimeDuration.java
#	iotdb-core/tsfile/src/test/java/org/apache/iotdb/tsfile/utils/TimeDurationTest.java
@JackieTien97 JackieTien97 merged commit f98b3d2 into apache:master Dec 18, 2023
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