-
Notifications
You must be signed in to change notification settings - Fork 3.4k
HBASE-29831 Fix for NPE in region replication (branch-2.5 backport) #7684
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-29831 Fix for NPE in region replication (branch-2.5 backport) #7684
Conversation
Signed-off-by: Chandra Kambham <[email protected]> Signed-off-by: Peng Lu <[email protected]>
|
🎊 +1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
There is a test case failed ( |
guluo2016
left a comment
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.
Thanks!
| // they come online. | ||
| if (useCache && locations.size() == 1 && TableName.isMetaTableName(tableName)) { | ||
| if (tableDescriptors.get(tableName).getRegionReplication() > 1) { | ||
| TableDescriptor td = tableDescriptors.get(tableName); |
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.
Sorry, I just noticed a few additional concerns after approving
I found the issue doesn't seem to exist for branch-2.5.
The code is only executed when seCache && locations.size() == 1 && TableName.isMetaTableName(tableName) is true, whch means that it's meta table, and the td of meta table is not null in here . right ? @krconv
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.
Oh I see; agreed it doesn't seem to apply here. Thank you for finding that! I'll close this.
Backport to branch-2.5 for PR #7629