-
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-23561 acquire lock before update regionsMap #1263
Conversation
💔 -1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
What is going on here @mwkang ? This should be a new JIRA? You have attached this to HBASE-23561, an issue that is closed. Thanks. |
Is this issue related to HBASE-23561? It seems so. Its like an addendum? Maybe it belongs on HBASE-23561 then? |
@saintstack I'm sorry if I did something wrong. I did patch HBASE-23561 but I added it because I thought it would be a problem later. Note Link: #1193 (comment) |
Nothing wrong done. Was having trouble figuring this an addendum and not some new work. Thanks. |
thank you for your review and comment. |
Addendum: tie add and delete of regionMap and encodedRegionsMap Signed-off-by: stack <stack@apache.org>
Addendum: tie add and delete of regionMap and encodedRegionsMap Signed-off-by: stack <stack@apache.org>
Addendum: tie add and delete of regionMap and encodedRegionsMap Signed-off-by: stack <stack@apache.org>
Addendum: tie add and delete of regionMap and encodedRegionsMap Signed-off-by: stack <stack@apache.org>
Addendum: tie add and delete of regionMap and encodedRegionsMap Signed-off-by: stack <stack@apache.org>
Addendum: tie add and delete of regionMap and encodedRegionsMap Signed-off-by: stack <stack@apache.org>
Addendum: tie add and delete of regionMap and encodedRegionsMap Signed-off-by: stack <stack@apache.org>
computeIfAbsent
not guaranteed to be applied once atomically. therefore they can have a different value.