caroliney14 commented on pull request #2769: URL: https://github.com/apache/hbase/pull/2769#issuecomment-745713635
@Apache9 There is already an `online` flag which is set in the RS after RS finishes all its setup work in `handleReportForDuty`, which essentially does what you are suggesting. However, currently RS does not actively communicate this flag to Master. Are you suggesting RS to communicate to Master saying its `online` flag is true, *after* the setup work in `reportForDuty`? Then my question is, do you know why we even do all this setup logic after we report to Master in the first place? Why not move the logic in `handleReportForDuty` to occur before we even report for duty to Master? I have not investigated the logic for this ordering so I don't know. But if there are no repercussions then we should just move the setup logic to occur before reporting to Master at all, as suggested in (1) [here](https://issues.apache.org/jira/browse/HBASE-25032?focusedCommentId=17241847&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-17241847). ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
