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

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

commit f6aec81748a179cc68ec4c459bf9c3a5f58a753b
Author: tkhurana <[email protected]>
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 <[email protected]>
---
 .../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 ac17884fe3..dcc82edb74 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 =

Reply via email to