-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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.
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
HBASE-26649 Support meta replica LoadBalance mode for RegionLocator#g… #4442
Conversation
Hi @Apache9, retry with a new patch, can you review? Thanks. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
I run locally the failed tests, they are ok. The test should not have anything related with the patch, let me rerun. |
💔 -1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
if (metaReplicaMode == CatalogReplicaMode.LOAD_BALANCE) { | ||
addListener(metaTable.getDescriptor(), (desc, error) -> { | ||
if (error != null) { | ||
LOG.warn("Failed to get meta table descriptotr, error={}", error); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here we do not want to log the stacktrace so we use error={}?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need the stacktrace, update in the new patch.
hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncNonMetaRegionLocator.java
Show resolved
Hide resolved
@@ -385,6 +386,12 @@ private void primaryNoChangeReplicaIncrease(final long[] before, final long[] af | |||
} | |||
} | |||
|
|||
private void PrimaryIncreaseReplicaIncrease(final long[] before, final long[] after) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the leading char should be lower case :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
.../java/org/apache/hadoop/hbase/replication/regionserver/TestMetaRegionReplicaReplication.java
Show resolved
Hide resolved
…etAllRegionLocations()
db1d9bb
to
1ea3008
Compare
💔 -1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
Thanks @Apache9 for the review. |
Please push an addendum to fix the spotless errors? Thanks. |
…etAllRegionLocations() (apache#4442) Signed-off-by: Duo Zhang <zhangduo@apache.org>
…etAllRegionLocations() (apache#4442) Signed-off-by: Duo Zhang <zhangduo@apache.org>
|
…etAllRegionLocations() (apache#4442) Signed-off-by: Duo Zhang <zhangduo@apache.org>
…etAllRegionLocations() (apache#4442) Signed-off-by: Duo Zhang <zhangduo@apache.org>
…etAllRegionLocations() (apache#4442) Signed-off-by: Duo Zhang <zhangduo@apache.org>
…etAllRegionLocations() (apache#4442) Signed-off-by: Duo Zhang <zhangduo@apache.org>
…etAllRegionLocations() (apache#4442) Signed-off-by: Duo Zhang <zhangduo@apache.org>
…ocator#getAllRegionLocations() (apache#4442)" This reverts commit 33041da.
…etAllRegionLocations() (apache#4442) (apache#4485) Signed-off-by: Duo Zhang <zhangduo@apache.org> Change-Id: Ic784e85e153325f91b88a1b8aa7ad25d041eeb78
…etAllRegionLocations()