Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/13285 )
Change subject: IMPALA-8528: Refactor authorization check in AnalysisContext ...................................................................... Patch Set 7: (2 comments) > Patch Set 6: > > (2 comments) > > Not quite following the pre/post/context stuff. I'm guessing this will be > used for audit? Probably cleaner to just keep this to a refactor (moving code > around) and not also introduce this new "context" concept until it's needed > for audits? Yeah I will introduce the new "context" concept until it's needed for audits in the later CR. http://gerrit.cloudera.org:8080/#/c/13285/6/fe/src/main/java/org/apache/impala/analysis/AnalysisContext.java File fe/src/main/java/org/apache/impala/analysis/AnalysisContext.java: http://gerrit.cloudera.org:8080/#/c/13285/6/fe/src/main/java/org/apache/impala/analysis/AnalysisContext.java@423 PS6, Line 423: authException = e; > I'm not entirely following the purpose of the pre/post stuff here. Couldn't Yeah this will be needed for the Ranger audit handler, but yeah I will introduce this class in the ranger audit handler CR instead. Done. http://gerrit.cloudera.org:8080/#/c/13285/6/fe/src/main/java/org/apache/impala/authorization/AuthorizationChecker.java File fe/src/main/java/org/apache/impala/authorization/AuthorizationChecker.java: http://gerrit.cloudera.org:8080/#/c/13285/6/fe/src/main/java/org/apache/impala/authorization/AuthorizationChecker.java@38 PS6, Line 38: * Authorize an analyzed statement. > this seems to be a bit more in that it's also supposed to return an authori Removed the pre/post methods in this CR. Done. -- To view, visit http://gerrit.cloudera.org:8080/13285 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I3bc3a11220dae0f49ef3e73d9ff27a90e9d4a71c Gerrit-Change-Number: 13285 Gerrit-PatchSet: 7 Gerrit-Owner: Fredy Wijaya <[email protected]> Gerrit-Reviewer: Austin Nobis <[email protected]> Gerrit-Reviewer: Fredy Wijaya <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Comment-Date: Mon, 13 May 2019 22:16:47 +0000 Gerrit-HasComments: Yes
