bbeaudreault commented on PR #5461:
URL: https://github.com/apache/hbase/pull/5461#issuecomment-1759553079

   > This problem only affects 2.x? Master and branch-3 are safe?
   
   It looks like master was fixed via your commit 
https://github.com/apache/hbase/commit/5d872d34225157bfbc18a6600c958c93d96636fd.
   
   Should we make a similar change here? Otherwise if we intend to just fix 
this bug here, then looking at ServerManager.getRsAdmin, it is not thread safe. 
We should probably use computeIfAbsent if the get returns null.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to