(phoenix) branch 5.2 updated: PHOENIX-7387 SnapshotScanner's next method is ignoring the boolean value from hbase's nextRaw method (#1971)

[stoty]

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

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


The following commit(s) were added to refs/heads/5.2 by this push:
     new 2a7a5a9c6e PHOENIX-7387 SnapshotScanner's next method is ignoring the 
boolean value from hbase's nextRaw method (#1971)
2a7a5a9c6e is described below

commit 2a7a5a9c6ef6bdedafb7cb786be780c737a5fc58
Author: Abhradeep Kundu <abhradeep.ku...@gmail.com>
AuthorDate: Wed Sep 4 16:53:07 2024 +0530

    PHOENIX-7387 SnapshotScanner's next method is ignoring the boolean value 
from hbase's nextRaw method (#1971)
    
    Co-authored-by: abhradeep.kundu <abhradeep.ku...@cloudera.com>
---
 .../src/main/java/org/apache/phoenix/iterate/SnapshotScanner.java | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git 
a/phoenix-core-server/src/main/java/org/apache/phoenix/iterate/SnapshotScanner.java
 
b/phoenix-core-server/src/main/java/org/apache/phoenix/iterate/SnapshotScanner.java
index 1a98515ba0..992ee46a7d 100644
--- 
a/phoenix-core-server/src/main/java/org/apache/phoenix/iterate/SnapshotScanner.java
+++ 
b/phoenix-core-server/src/main/java/org/apache/phoenix/iterate/SnapshotScanner.java
@@ -160,14 +160,14 @@ public class SnapshotScanner extends 
AbstractClientScanner {
   @Override
   public Result next() throws IOException {
     values.clear();
-    scanner.nextRaw(values);
+    boolean hasMore = scanner.nextRaw(values);
     statisticsCollector.collectStatistics(values);
-    if (values.isEmpty()) {
+    if (hasMore || !values.isEmpty()) {
+      return Result.create(values);
+    } else {
       //we are done
       return null;
     }
-
-    return Result.create(values);
   }
 
   @Override