virajjasani commented on pull request #2172: URL: https://github.com/apache/hbase/pull/2172#issuecomment-666528112
> is there a particular reason we're conflating formatting changes here? > > are the qa failures related? QA failures are not relevant, tried locally and both builds have one common test failure: `TestReplicationEndpointWithMultipleAsyncWAL`, works quite well locally with the patch. I just thought of refactoring `MoveWithAck` and `MoveWithoutAck` as high level classes since RegionMover is quite huge. ---------------------------------------------------------------- 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: us...@infra.apache.org
