liuzhongjun89 commented on PR #4950: URL: https://github.com/apache/hbase/pull/4950#issuecomment-1377186951
Think you for review, Exactly as you said, The old code just lose previous region info when the next table is disabled. In the scenario you mentioned, ReportMakingVisitor will keep the previous region info and check it on visitor close, just like it is the last table > So the problem here is that, if the table is disabled, we will skip checking but in the old code, we will still record the previous region info, so the check for the previous table's last region will be skipped? > > The fix here is to not record the previous region for disabled table? What if there are only two tables, one is enabled and we check it first, the other is disabled? Do we still have chance to check the last region for the enabled table? -- 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]
