[GitHub] [hbase] bharathv commented on issue #798: HBASE-23257: Track clusterID in stand by masters

2019-11-06 Thread GitBox
bharathv commented on issue #798: HBASE-23257: Track clusterID in stand by masters URL: https://github.com/apache/hbase/pull/798#issuecomment-550577163 @apurtell lol, race condition while reviewing / pushing. Thanks for taking a look :-) ---

[GitHub] [hbase] bharathv commented on issue #798: HBASE-23257: Track clusterID in stand by masters

2019-11-06 Thread GitBox
bharathv commented on issue #798: HBASE-23257: Track clusterID in stand by masters URL: https://github.com/apache/hbase/pull/798#issuecomment-550576975 Re-did the patch slightly. Got rid of RW locks, refactored for better readability. Let me know what you think. --

[GitHub] [hbase] bharathv commented on issue #798: HBASE-23257: Track clusterID in stand by masters

2019-11-06 Thread GitBox
bharathv commented on issue #798: HBASE-23257: Track clusterID in stand by masters URL: https://github.com/apache/hbase/pull/798#issuecomment-550421694 Fixed the check-style nits. Test failures don't seem related to the patch, triggering another run just to make sure.

[GitHub] [hbase] bharathv commented on issue #798: HBASE-23257: Track clusterID in stand by masters

2019-11-05 Thread GitBox
bharathv commented on issue #798: HBASE-23257: Track clusterID in stand by masters URL: https://github.com/apache/hbase/pull/798#issuecomment-550107231 @apurtell @wchevreuil FYI. I split the change into smaller tasks for easier reviews. This one adds stand-by cluster ID caching. -