NihalJain commented on PR #7545:
URL: https://github.com/apache/hbase/pull/7545#issuecomment-3673860886

   > Hi @NihalJain, IMO `TestDebugDumpRedaction` tests the 
`StateDumpServlet.getRedactedConfiguration()` pretty thoroughly, so I did not 
move the method and add individual tests. Thanks for approving the PR. Let me 
know if you decide I should add individual tests for that as well. Otherwise, 
it looks like this PR can be merged.
   
   Hey lgtm, we are good here


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