ctubbsii edited a comment on pull request #1798: URL: https://github.com/apache/accumulo/pull/1798#issuecomment-740740073
> This looks great to me. I did some manual testing, and everything works as expected. I do think it would be useful to see an INFO (maybe DEBUG? any opinions here?) message when each user's credentials are upgraded, though. Don't worry about this one. I will add a commit to add a log message for the upgrade action before merging. Everything here looks great! I will merge it soon. Thanks, @BukrosSzabolcs ! > Also, I noticed some strange "denied" messages for "performSystemAction" when a user successfully authenticates. I don't know if that's new with this PR or not. I'm going to see if I can track it down. If it's pre-existing, then I wouldn't worry about it. It looks like this is a pre-existing issue. Don't worry about it. (EDIT: fixed in #1827 ) ---------------------------------------------------------------- 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: [email protected]
