You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@iotdb.apache.org by qi...@apache.org on 2021/07/30 13:13:42 UTC

[iotdb] branch rel/0.11 updated: Fix order by time desc batchdata bug (#3652)

This is an automated email from the ASF dual-hosted git repository.

qiaojialin pushed a commit to branch rel/0.11
in repository https://gitbox.apache.org/repos/asf/iotdb.git


The following commit(s) were added to refs/heads/rel/0.11 by this push:
     new 9d095c8  Fix order by time desc batchdata bug (#3652)
9d095c8 is described below

commit 9d095c885ff38c2ae753cc2d4a02fa03c7c9f782
Author: Xiangwei Wei <34...@users.noreply.github.com>
AuthorDate: Fri Jul 30 21:13:23 2021 +0800

    Fix order by time desc batchdata bug (#3652)
---
 .../main/java/org/apache/iotdb/db/query/reader/series/SeriesReader.java | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/server/src/main/java/org/apache/iotdb/db/query/reader/series/SeriesReader.java b/server/src/main/java/org/apache/iotdb/db/query/reader/series/SeriesReader.java
index 87fa80a..dc48e8f 100644
--- a/server/src/main/java/org/apache/iotdb/db/query/reader/series/SeriesReader.java
+++ b/server/src/main/java/org/apache/iotdb/db/query/reader/series/SeriesReader.java
@@ -628,6 +628,7 @@ public class SeriesReader {
             if ((orderUtils.getAscending() && timeValuePair.getTimestamp() > firstPageReader
                 .getStatistics().getEndTime()) || (!orderUtils.getAscending()
                 && timeValuePair.getTimestamp() < firstPageReader.getStatistics().getStartTime())) {
+              cachedBatchData.flip();
               hasCachedNextOverlappedPage = cachedBatchData.hasCurrent();
               return hasCachedNextOverlappedPage;
             } else {
@@ -646,6 +647,7 @@ public class SeriesReader {
                 .getStatistics().getEndTime()) || (!orderUtils.getAscending()
                 && timeValuePair.getTimestamp() < seqPageReaders.get(0).getStatistics()
                 .getStartTime())) {
+              cachedBatchData.flip();
               hasCachedNextOverlappedPage = cachedBatchData.hasCurrent();
               return hasCachedNextOverlappedPage;
             } else {