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

[iotdb] 01/01: Fix order by time desc batchdata bug

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

xiangweiwei pushed a commit to branch descBatchDataBug
in repository https://gitbox.apache.org/repos/asf/iotdb.git

commit 1266470798d11ddf9dec567c0c21c89b6d89f086
Author: Alima777 <wx...@gmail.com>
AuthorDate: Fri Jul 30 21:01:00 2021 +0800

    Fix order by time desc batchdata bug
---
 .../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 {