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:19 UTC

[iotdb] branch descBatchDataBug created (now 1266470)

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

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


      at 1266470  Fix order by time desc batchdata bug

This branch includes the following new commits:

     new 1266470  Fix order by time desc batchdata bug

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


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

Posted by xi...@apache.org.
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 {