dlmarion commented on pull request #142:
URL: https://github.com/apache/accumulo-testing/pull/142#issuecomment-872154221


   This looks ok to me. I tried to find a class in the `accumulo` repo that 
could be used in place of KerberosHelper, but I'm not sure one exists. My only 
question is "should all of the classes in `accumulo-testing` be set up in the 
same manner, to enable them to run in an environment running Kerberos?". I 
think the answer is yes, but it could be done after this change, maybe a good 
first issue ticket.


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