Skip to content

BIGTOP-4527. Upgrade Phoenix to 5.3.0. #1387

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
iwasakims wants to merge 1 commit into
base: master
Choose a base branch
from
+32 −1
Open

BIGTOP-4527. Upgrade Phoenix to 5.3.0. #1387

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
31 changes: 31 additions & 0 deletions bigtop-packages/src/common/phoenix/patch1-PHOENIX-7741.diff
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,31 @@
commit 3de9b9eb2e4be9e2763bb2932e45d025c90f0c20
Author: Umesh <9414umeshkumar@gmail.com>
Date: Wed Dec 10 08:55:51 2025 +0530

PHOENIX-7741 Remove usage of isRegionInTransition method from phoenix tests (#2330)

---------

Co-authored-by: ukumawat <ukumawat@salesforce.com>
(cherry picked from commit fd6a76e66f04718d9ec4aaa142ca43ebf6c8cad8)

diff --git a/phoenix-core/src/it/java/org/apache/phoenix/rpc/PhoenixServerRpcIT.java b/phoenix-core/src/it/java/org/apache/phoenix/rpc/PhoenixServerRpcIT.java
index 2c1752ba02..3dc8238855 100644
--- a/phoenix-core/src/it/java/org/apache/phoenix/rpc/PhoenixServerRpcIT.java
+++ b/phoenix-core/src/it/java/org/apache/phoenix/rpc/PhoenixServerRpcIT.java
@@ -243,11 +243,14 @@ public class PhoenixServerRpcIT extends BaseTest {
}
byte[] encodedRegionNameInBytes = hri2.getEncodedNameAsBytes();
admin.move(encodedRegionNameInBytes, dstServer.getServerName());
+ // verify that both HMaster and Region server are aware of this movement
while (
dstServer.getOnlineRegion(hri2.getRegionName()) == null
|| dstServer.getRegionsInTransitionInRS().containsKey(encodedRegionNameInBytes)
|| srcServer.getRegionsInTransitionInRS().containsKey(encodedRegionNameInBytes)
- || master.getAssignmentManager().getRegionStates().isRegionInTransition(hri2)
+ || !ServerName.isSameAddress(
+ master.getAssignmentManager().getRegionStates().getRegionServerOfRegion(hri2),
+ dstServer.getServerName())
) {
// wait for the move to be finished
Thread.sleep(1);
2 changes: 1 addition & 1 deletion bigtop.bom
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -247,7 +247,7 @@ bigtop {
'phoenix' {
name = 'phoenix'
relNotes = 'Apache Phoenix: A SQL skin over HBase'
version { base = "5.2.1"; pkg = base; release = 1 }
version { base = "5.3.0"; pkg = base; release = 1 }
tarball { destination = "$name-${version.base}-src.tar.gz"
source = destination }
url { download_path = "/$name/$name-${version.base}"
Expand Down