ctubbsii opened a new pull request, #5483: URL: https://github.com/apache/accumulo/pull/5483
* Revert the new log message added to the ZKAuthorizor in 3825d582e7ddecb2d3661f96a707c13120bec609 for #5480 because the information is already available to the users by another means, and it's not worth the risk of bad user requests spamming the server logs * Remove a similar extraneous exception and try-catch block for getActiveScans method that was removed in #5480 * Improve exception handling of TabletClientHandler.checkPermission by separating out the nested try-throw-catch of ThriftSecurityException into separate steps, so that the correct messages can be logged when the exception occurs due to permission being denied, rather than due to an unauthenticatable user (previously, the log message would always say it was from an unauthenticatable user, which is not true); this also removes the redundant logging at the trace level when the message was already logged differently -- 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]
