kgeisz commented on PR #7417:
URL: https://github.com/apache/hbase/pull/7417#issuecomment-3487908636

   I made another small change to this PR.  The original 
IntegrationTestBackupRestore file had a bug in how it configures command line 
parameters.  In `IntegrationTestBackupRestore` on the master branch, the 
`processOptions()` method is doing proper initialization.  Later, in the 
`setUp()` method, [this block of 
code](https://github.com/apache/hbase/blob/master/hbase-it/src/test/java/org/apache/hadoop/hbase/IntegrationTestBackupRestore.java#L142-L147)
 overwrites these values.  I was doing the same in my PR, but now I have 
realized I should not.


-- 
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]

Reply via email to