nirdosh0110 commented on PR #8583: URL: https://github.com/apache/hbase/pull/8583#issuecomment-5684015682
> > What is the order for PR merges we are following for the relevant work @Apache9 @apurtell? > > > > * [HBASE-30334: Reopen parent regions on rollback of MERGE_TABLE_REGIONS_CHECK_CLOSED_REGIONS #8583](https://github.com/apache/hbase/pull/8583) > > * [HBASE-30335 Seed master flushedSequenceIdByRegion with openSeqNum on region OPEN #8584](https://github.com/apache/hbase/pull/8584) > > * [HBASE-30352: Tolerate stale recovered.edits below durable seqid in split/merge #8599](https://github.com/apache/hbase/pull/8599) > @nirdosh0110 what is the sequence of PRs to check-in one by one? Hi @virajjasani, - We'd need one more review approval on #8599 before merging it in. - There are no compile/test dependencies between the three PRs — they touch different files and different subsystems. However, there is a clear safety ordering we should follow. Suggested merge order: 1. #8583 — Rollback fix; prevents stuck RITs. 2. #8599 — Tolerate stale recovered.edits; provides immediate prod relief. Blocker: resolve Apache9's HFile metadata request first. 3. #8584 — Seed durability state on OPEN; addresses the root cause. This is the largest change, so it should be merged last. Let me know your thoughts. 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]
