kgyrtkirk commented on a change in pull request #2033:
URL: https://github.com/apache/hive/pull/2033#discussion_r585539220
##########
File path: ql/src/java/org/apache/hadoop/hive/ql/exec/ExplainTask.java
##########
@@ -541,12 +606,12 @@ JSONObject collectAuthRelatedEntities(PrintStream out,
ExplainWork work)
if (delegate != null) {
Class itface = SessionState.get().getAuthorizerInterface();
Object authorizer = AuthorizationFactory.create(delegate, itface,
- new AuthorizationFactory.AuthorizationExceptionHandler() {
- @Override
- public void exception(Exception exception) {
- exceptions.add(exception.getMessage());
- }
- });
+ new
AuthorizationFactory.AuthorizationExceptionHandler() {
+ @Override
Review comment:
there are lots of pure indentation changes in this patch - are we using
the same formatter settings?
`dev-support/eclipse-styles.xml`
----------------------------------------------------------------
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
