sidkhillon opened a new pull request, #8858: URL: https://github.com/apache/hbase/pull/8858
When a replication peer is switched from non-serial to serial, `ModifyPeerProcedure.reopenRegions` reopens every region of the peer's tables with `new ReopenTableRegionsProcedure(tn)`. That constructor turns batching off, so `hbase.reopen.table.regions.progressive.batch.size.max` and `hbase.reopen.table.regions.progressive.batch.backoff.ms` are ignored and every region of the table closes and reopens at the same time. The same happens when a serial peer is added for tables that already have global replication scope. ### Testing Added `TestSerialPeerReopenThrottling`. It sets batch size max to 1, creates a 5-region table with global replication scope, adds a non-serial peer, switches it to serial, and checks that the reopen ran 5 batches that reopened 5 regions. It fails without the fix. -- 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]
