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

palashc 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 510c8a7f9b PHOENIX-7578 : Fix CDCStreamIT tests with region merges 
(#2104)
510c8a7f9b is described below

commit 510c8a7f9b07be42c244bcb7b9fa484a5d949adb
Author: Palash Chauhan <palashc...@gmail.com>
AuthorDate: Thu Apr 10 11:51:56 2025 -0700

    PHOENIX-7578 : Fix CDCStreamIT tests with region merges (#2104)
    
    Co-authored-by: Palash Chauhan 
<p.chau...@pchauha-ltmgv47.internal.salesforce.com>
---
 phoenix-core/src/it/java/org/apache/phoenix/end2end/CDCStreamIT.java | 2 ++
 phoenix-core/src/test/java/org/apache/phoenix/util/TestUtil.java     | 2 +-
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git 
a/phoenix-core/src/it/java/org/apache/phoenix/end2end/CDCStreamIT.java 
b/phoenix-core/src/it/java/org/apache/phoenix/end2end/CDCStreamIT.java
index 98dd22f535..5061efe31b 100644
--- a/phoenix-core/src/it/java/org/apache/phoenix/end2end/CDCStreamIT.java
+++ b/phoenix-core/src/it/java/org/apache/phoenix/end2end/CDCStreamIT.java
@@ -445,6 +445,8 @@ public class CDCStreamIT extends CDCBaseIT {
                 .map(HRegionLocation::getRegion)
                 .map(RegionInfo::getEncodedName)
                 .collect(Collectors.toList()));
+        // compact to remove merge qualifier from merged regions i.e. clear 
references to parents
+        TestUtil.doMajorCompaction(conn, tableName);
         // merge the two regions
         regions = TestUtil.getAllTableRegions(conn, tableName);
         TestUtil.mergeTableRegions(conn, tableName, regions.stream()
diff --git a/phoenix-core/src/test/java/org/apache/phoenix/util/TestUtil.java 
b/phoenix-core/src/test/java/org/apache/phoenix/util/TestUtil.java
index 5576fcc216..784e8439f3 100644
--- a/phoenix-core/src/test/java/org/apache/phoenix/util/TestUtil.java
+++ b/phoenix-core/src/test/java/org/apache/phoenix/util/TestUtil.java
@@ -1533,7 +1533,7 @@ public class TestUtil {
                 .toArray(byte[][]::new);
 
         executeHBaseTableRegionOperation(conn, tableName, (admin, 
regionLocator, nRegions) -> {
-            admin.mergeRegionsAsync(regionsToMerge, true);
+            admin.mergeRegionsAsync(regionsToMerge, true).get();
             waitForRegionChange(regionLocator, nRegions);
         });
     }

Reply via email to