----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23527/#review47953 -----------------------------------------------------------
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLStdHiveAuthorizationValidator.java <https://reviews.apache.org/r/23527/#comment84196> Is this the only current usage of the context info? Should it be logged for failed auth checks? - Jason Dere On July 15, 2014, 10:48 p.m., Thejas Nair wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/23527/ > ----------------------------------------------------------- > > (Updated July 15, 2014, 10:48 p.m.) > > > Review request for hive. > > > Bugs: HIVE-7416 > https://issues.apache.org/jira/browse/HIVE-7416 > > > Repository: hive-git > > > Description > ------- > > See jira > > > Diffs > ----- > > > itests/hive-unit/src/test/java/org/apache/hive/jdbc/authorization/TestHS2AuthzContext.java > PRE-CREATION > ql/src/java/org/apache/hadoop/hive/ql/Driver.java ac76214 > ql/src/java/org/apache/hadoop/hive/ql/exec/ExplainTask.java 92545d8 > > ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAuthorizationValidator.java > 7ffbc44 > > ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAuthorizer.java > dbef61a > > ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAuthorizerImpl.java > 558d4ff > > ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAuthzContext.java > PRE-CREATION > > ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLStdHiveAuthorizationValidator.java > 8937cfa > ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java 6686bc6 > service/src/java/org/apache/hive/service/cli/session/HiveSessionImpl.java > 6a7ee7a > > Diff: https://reviews.apache.org/r/23527/diff/ > > > Testing > ------- > > New tests included. > > > Thanks, > > Thejas Nair > >