----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17163/#review32517 -----------------------------------------------------------
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAuthorizationPluginException.java <https://reviews.apache.org/r/17163/#comment61384> If we make it extend HiveException than we can avoid all those try-catches which catch this exception and rethrow HiveException. ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLStdHiveAuthorizationValidator.java <https://reviews.apache.org/r/17163/#comment61385> This can be removed. ql/src/test/queries/clientpositive/authorization_1_sql_std.q <https://reviews.apache.org/r/17163/#comment61392> If you don't intent to have this in test case, better to just delete it. ql/src/test/queries/clientpositive/authorization_1_sql_std.q <https://reviews.apache.org/r/17163/#comment61393> Same as above. - Ashutosh Chauhan On Jan. 22, 2014, 12:17 a.m., Thejas Nair wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/17163/ > ----------------------------------------------------------- > > (Updated Jan. 22, 2014, 12:17 a.m.) > > > Review request for hive and Ashutosh Chauhan. > > > Bugs: HIVE-5929 > https://issues.apache.org/jira/browse/HIVE-5929 > > > Repository: hive-git > > > Description > ------- > > Subtask for sql standard based auth, for performing the updates to metastore > from newly supported access control statements . > > > Diffs > ----- > > ql/src/java/org/apache/hadoop/hive/ql/Driver.java bd95161 > ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java 92ed55b > ql/src/java/org/apache/hadoop/hive/ql/metadata/HiveUtils.java c65bf28 > > ql/src/java/org/apache/hadoop/hive/ql/parse/authorization/HiveAuthorizationTaskFactoryImpl.java > c41cd0f > > ql/src/java/org/apache/hadoop/hive/ql/security/authorization/AuthorizationUtils.java > PRE-CREATION > > ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/DefaultHiveAuthorizerFactory.java > 7470e9d > > ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAccessController.java > 8e4114f > > ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAuthorizationPluginException.java > PRE-CREATION > > ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAuthorizationValidator.java > 63046f5 > > ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAuthorizer.java > c10a2ac > > ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAuthorizerImpl.java > ca95bfc > > ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HivePrivilegeInfo.java > PRE-CREATION > > ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLStdHiveAccessController.java > PRE-CREATION > > ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLStdHiveAuthorizationValidator.java > PRE-CREATION > > ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLStdHiveAuthorizerFactory.java > PRE-CREATION > ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java 08f7fec > ql/src/test/queries/clientpositive/authorization_1_sql_std.q PRE-CREATION > ql/src/test/queries/clientpositive/authorization_role_grant1.q 1a375a5 > ql/src/test/results/clientpositive/authorization_1_sql_std.q.out > PRE-CREATION > ql/src/test/results/clientpositive/authorization_role_grant1.q.out 496687c > > Diff: https://reviews.apache.org/r/17163/diff/ > > > Testing > ------- > > .q tests included. > > > Thanks, > > Thejas Nair > >
