You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@phoenix.apache.org by st...@apache.org on 2023/11/29 08:56:01 UTC

(phoenix) branch master updated: PHOENIX-7109 Incorrect query results when using OFFSET

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

stoty pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/phoenix.git


The following commit(s) were added to refs/heads/master by this push:
     new 1a2f09ced1 PHOENIX-7109 Incorrect query results when using OFFSET
1a2f09ced1 is described below

commit 1a2f09ced11206b934367852c453dc825901578d
Author: Aron Meszaros <me...@gmail.com>
AuthorDate: Wed Nov 22 16:14:28 2023 +0100

    PHOENIX-7109 Incorrect query results when using OFFSET
---
 .../src/main/java/org/apache/phoenix/iterate/OffsetResultIterator.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/phoenix-core/src/main/java/org/apache/phoenix/iterate/OffsetResultIterator.java b/phoenix-core/src/main/java/org/apache/phoenix/iterate/OffsetResultIterator.java
index e00a9fbeb4..61de07500b 100644
--- a/phoenix-core/src/main/java/org/apache/phoenix/iterate/OffsetResultIterator.java
+++ b/phoenix-core/src/main/java/org/apache/phoenix/iterate/OffsetResultIterator.java
@@ -49,7 +49,7 @@ public class OffsetResultIterator extends DelegateResultIterator {
         while (rowCount < offset) {
             Tuple tuple = super.next();
             if (tuple == null) { return null; }
-            if (isDummy(tuple)) {
+            if (tuple.size() == 0 || isDummy(tuple)) {
                 // while rowCount < offset absorb the dummy and call next on the underlying scanner
                 continue;
             }