Umeshkumar9414 commented on PR #8584: URL: https://github.com/apache/hbase/pull/8584#issuecomment-5790451947
> Both CLOSE-time and OPEN-time seeding would fix this. Reasons I kept OPEN-time: > > * `openSeqNum` ≥ close-marker, so it's an at-least-as-tight fence. > * Fires on every OPEN (reassignment after SCP, master restart, RS restart), not only clean closes. > * Reuses the existing `reportRegionOpen` payload that already carries `openSeqNum` — no new field on the close report. But this will still leave a small gap where Close is completed and OPEN is yet to happen. And SCP of source is already started and we are about to split the wal. This is the reason why I wanted to do it with `reportRegionClose` . Because flush is something related to close only not open. So it would make sense if we do this while closing. -- 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]
