[phoenix] branch 4.x updated: PHOENIX-6356 missing row.clear() for dummy row in GlobalIndexRegionScanner

[st...@apache.org]

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

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


The following commit(s) were added to refs/heads/4.x by this push:
     new 7ffb59c  PHOENIX-6356 missing row.clear() for dummy row in GlobalIndexRegionScanner
7ffb59c is described below

commit 7ffb59c6a30b898ea0ccde1bd28c1b38ff7e5a49
Author: Tanuj Khurana <kh...@gmail.com>
AuthorDate: Wed Feb 3 17:50:22 2021 -0800

    PHOENIX-6356 missing row.clear() for dummy row in GlobalIndexRegionScanner
    
    If PagedScanFilter is set on the scan, a dummy row can be returned
    which needs to be skipped.
---
 .../java/org/apache/phoenix/coprocessor/GlobalIndexRegionScanner.java    | 1 +
 1 file changed, 1 insertion(+)

diff --git a/phoenix-core/src/main/java/org/apache/phoenix/coprocessor/GlobalIndexRegionScanner.java b/phoenix-core/src/main/java/org/apache/phoenix/coprocessor/GlobalIndexRegionScanner.java
index 540851a..6db1d4e 100644
--- a/phoenix-core/src/main/java/org/apache/phoenix/coprocessor/GlobalIndexRegionScanner.java
+++ b/phoenix-core/src/main/java/org/apache/phoenix/coprocessor/GlobalIndexRegionScanner.java
@@ -1452,6 +1452,7 @@ public abstract class GlobalIndexRegionScanner extends BaseRegionScanner {
                     hasMoreIncr = scanner.nextRaw(row);
                     if (!row.isEmpty()) {
                         if (isDummy(row)) {
+                            row.clear();
                             continue;
                         }
                         keys.add(PVarbinary.INSTANCE.getKeyRange(CellUtil.cloneRow(row.get(0))));