[GitHub] [hbase] bharathv commented on pull request #2769: HBASE-25032 Wait for region server to become online before adding it to online servers in Master

2021-03-22 Thread GitBox
bharathv commented on pull request #2769: URL: https://github.com/apache/hbase/pull/2769#issuecomment-804508669 Sorry don't mean to drag this PR, just want to make sure that we don't create test flakes. Otherwise there will be weird timeouts in the hbase mini cluster util that many downstr

[GitHub] [hbase] bharathv commented on pull request #2769: HBASE-25032 Wait for region server to become online before adding it to online servers in Master

2021-02-26 Thread GitBox
bharathv commented on pull request #2769: URL: https://github.com/apache/hbase/pull/2769#issuecomment-786777641 @caroliney14 Is this ready for review or are you just testing a patch? I don't see the test originally added in the PR. Also, a bunch of seemingly related tests [failed](https:/

[GitHub] [hbase] bharathv commented on pull request #2769: HBASE-25032 Wait for region server to become online before adding it to online servers in Master

2021-01-05 Thread GitBox
bharathv commented on pull request #2769: URL: https://github.com/apache/hbase/pull/2769#issuecomment-755028030 Stumbled upon this one. > Just a simple flag like initialized is enough? After we finish the initialization work, we set the flag to true, and regionServerReport will brin