Sigma-Ma commented on PR #8483:
URL: https://github.com/apache/hbase/pull/8483#issuecomment-5174072540

   Hi @wchevreuil and @ndimiduk, could you please take a look at this PR when 
convenient?
   
   This patch fixes the RegionServer startup race reported in HBASE-30099. 
During RegionServer initialization, `getBootstrapNodes()` may be called before 
`BootstrapNodeManager` has been created, which currently results in an NPE.
   
   The proposed fix makes the RPC return an empty bootstrap-node list during 
this short initialization window. Once `BootstrapNodeManager` is initialized, 
the existing behavior remains unchanged. A regression test has also been added 
to cover the RPC call before manager initialization.
   
   @ndimiduk, since you reported HBASE-30099, it would also be helpful if you 
could confirm whether returning an empty list during startup matches the 
expected behavior.
   
   The PR is currently mergeable and has not received a review yet. If the 
forked GitHub Actions workflows still require approval, could a committer 
please approve them as well?
   
   Thanks!


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to