----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16034/ -----------------------------------------------------------
(Updated April 2, 2014, 9:07 a.m.) Review request for hive. Changes ------- Rebased to trunk Bugs: HIVE-5961 https://issues.apache.org/jira/browse/HIVE-5961 Repository: hive-git Description ------- For easy checking of need privileges for a query, {noformat} explain authorize select * from src join srcpart INPUTS: default@srcpart default@srcpart@ds=2008-04-08/hr=11 default@srcpart@ds=2008-04-08/hr=12 default@srcpart@ds=2008-04-09/hr=11 default@srcpart@ds=2008-04-09/hr=12 default@src OUTPUTS: file:/home/navis/apache/oss-hive/itests/qtest/target/tmp/localscratchdir/hive_2013-12-04_21-57-53_748_5323811717799107868-1/-mr-10000 CURRENT_USER: hive_test_user OPERATION: QUERY AUTHORIZATION_FAILURES: No privilege 'Select' found for inputs { database:default, table:srcpart, columnName:key} No privilege 'Select' found for inputs { database:default, table:src, columnName:key} No privilege 'Select' found for inputs { database:default, table:src, columnName:key} {noformat} Hopefully good for debugging of authorization, which is in progress on HIVE-5837. Diffs (updated) ----- ql/src/java/org/apache/hadoop/hive/ql/Driver.java d42895a ql/src/java/org/apache/hadoop/hive/ql/exec/ExplainTask.java 35f4fa9 ql/src/java/org/apache/hadoop/hive/ql/parse/BaseSemanticAnalyzer.java db9fa74 ql/src/java/org/apache/hadoop/hive/ql/parse/ExplainSemanticAnalyzer.java 77fb8bd ql/src/java/org/apache/hadoop/hive/ql/parse/HiveLexer.g 3e673ca ql/src/java/org/apache/hadoop/hive/ql/parse/HiveParser.g 13bbf0a ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzerFactory.java e7d0359 ql/src/java/org/apache/hadoop/hive/ql/plan/ExplainWork.java d7140ca ql/src/java/org/apache/hadoop/hive/ql/security/authorization/AuthorizationFactory.java PRE-CREATION ql/src/java/org/apache/hadoop/hive/ql/security/authorization/DelegatableAuthorizationProvider.java PRE-CREATION ql/src/test/queries/clientpositive/authorization_explain.q PRE-CREATION ql/src/test/results/clientpositive/authorization_explain.q.out PRE-CREATION Diff: https://reviews.apache.org/r/16034/diff/ Testing ------- Thanks, Navis Ryu