PHOENIX-7870 :- Per-HA-group poller futures and url1/url2 alternation in GetClusterRoleRecordUtil (ADDENDUM)#2494
Merged
lokiore merged 1 commit intoJun 3, 2026
Conversation
lokiore
changed the title
… in GetClusterRoleRecordUtil (ADDENDUM) Pure formatting addendum produced by `mvn spotless:apply -DskipTests -q` on the current tip of PHOENIX-7562-feature-new. No source-logic changes. The original PHOENIX-7870 merge into PHOENIX-7562-feature-new shipped without `spotless:apply` being run pre-PR, so the feature branch is currently failing `spotless:check`. This addendum brings it back in compliance. Files affected (7): - phoenix-core-client/.../exception/MutationBlockedIOException.java - phoenix-core-client/.../exception/StaleClusterRoleRecordException.java - phoenix-core-client/.../schema/MetaDataClient.java - phoenix-core-client/.../util/GetClusterRoleRecordUtil.java - phoenix-core/.../end2end/index/IndexRegionObserverMutationBlockingIT.java - phoenix-core/.../exception/HaFailoverExceptionInheritanceTest.java - phoenix-core/.../util/GetClusterRoleRecordUtilTest.java Local verification: - mvn spotless:check on the tree post-apply: PASS - mvn -pl phoenix-core-client -am compile: PASS - mvn -pl phoenix-core-client -am test-compile: PASS - mvn -pl phoenix-core -am test-compile: PASS - mvn -pl phoenix-core-client -am test -Dtest=GetClusterRoleRecordUtilTest: PASS Generated-by: Claude Code (Opus 4.7)
lokiore
force-pushed
the
PHOENIX-7870-spotless-addendum
branch
from
22d53c1 to
07fc41c
Compare
tkhurana
approved these changes
Jun 3, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Pure-formatting addendum on top of PHOENIX-7562-feature-new produced by
mvn spotless:apply -DskipTests -q. No source-logic changes.The earlier merges into PHOENIX-7562-feature-new (PHOENIX-7870 / PHOENIX-7871 / preceding) inadvertently shipped without
spotless:applybeing run pre-PR, so the feature branch is currently failingspotless:check. This PR brings the branch back in compliance.What changed
7 files (Javadoc rewraps,
<p>tag placement, line-length-driven argument breaks):phoenix-core-client/src/main/java/org/apache/phoenix/exception/MutationBlockedIOException.javaphoenix-core-client/src/main/java/org/apache/phoenix/exception/StaleClusterRoleRecordException.javaphoenix-core-client/src/main/java/org/apache/phoenix/schema/MetaDataClient.javaphoenix-core-client/src/main/java/org/apache/phoenix/util/GetClusterRoleRecordUtil.javaphoenix-core/src/it/java/org/apache/phoenix/end2end/index/IndexRegionObserverMutationBlockingIT.javaphoenix-core/src/test/java/org/apache/phoenix/exception/HaFailoverExceptionInheritanceTest.javaphoenix-core/src/test/java/org/apache/phoenix/util/GetClusterRoleRecordUtilTest.javaDiff stat: 102 insertions(+), 107 deletions(-).
git diff -wshows zero non-whitespace delta.Test plan
mvn spotless:apply -DskipTests -qproduced the diff (deterministic)mvn spotless:check -DskipTests -qPASSES on the resulting treemvn -pl phoenix-core-client -am compilePASSmvn -pl phoenix-core-client -am test-compilePASSmvn -pl phoenix-core -am test-compilePASS (for the IT module that owns one of the modified files)mvn -pl phoenix-core-client -am test -Dtest=GetClusterRoleRecordUtilTestPASSAI disclosure
Part of this work was produced with the assistance of Claude Code (Opus 4.7), an AI coding assistant. A human contributor reviewed all changes before committing and is responsible for the result. Per the ASF generative-tooling policy, the commit also carries a
Generated-by: Claude Code (Opus 4.7)trailer.Backwards compatibility
No behavioral change — formatting only.
Migration notes
None.