(phoenix) branch 5.2 updated: PHOENIX-7295 Fix getTableRegions failing due to overlap/inconsistencies on region (#1870)

[vj...@apache.org]

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

vjasani 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 2aa45d8e56 PHOENIX-7295 Fix getTableRegions failing due to overlap/inconsistencies on region (#1870)
2aa45d8e56 is described below

commit 2aa45d8e564575a75a7145ef7ae5df8b7788c5ba
Author: tkhurana <kh...@gmail.com>
AuthorDate: Tue Apr 2 16:07:49 2024 -0700

    PHOENIX-7295 Fix getTableRegions failing due to overlap/inconsistencies on region (#1870)
    
    Co-authored-by: Tanuj Khurana <tk...@apache.org>
---
 .../main/java/org/apache/phoenix/query/ConnectionQueryServicesImpl.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/phoenix-core-client/src/main/java/org/apache/phoenix/query/ConnectionQueryServicesImpl.java b/phoenix-core-client/src/main/java/org/apache/phoenix/query/ConnectionQueryServicesImpl.java
index 22076a2497..4578b075e2 100644
--- a/phoenix-core-client/src/main/java/org/apache/phoenix/query/ConnectionQueryServicesImpl.java
+++ b/phoenix-core-client/src/main/java/org/apache/phoenix/query/ConnectionQueryServicesImpl.java
@@ -780,13 +780,13 @@ public class ConnectionQueryServicesImpl extends DelegateQueryServices implement
             config.getInt(PHOENIX_GET_REGIONS_RETRIES, DEFAULT_PHOENIX_GET_REGIONS_RETRIES);
         TableName table = TableName.valueOf(tableName);
         byte[] currentKey = null;
-        HRegionLocation prevRegionLocation = null;
         while (true) {
             try {
                 // We could surface the package projected HConnectionImplementation.getNumberOfCachedRegionLocations
                 // to get the sizing info we need, but this would require a new class in the same package and a cast
                 // to this implementation class, so it's probably not worth it.
                 List<HRegionLocation> locations = Lists.newArrayList();
+                HRegionLocation prevRegionLocation = null;
                 currentKey = startRowKey;
                 do {
                     HRegionLocation regionLocation =