----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16565/#review31042 -----------------------------------------------------------
LGTM. Is this ready for commit? Also, can you check out HIVE-5946? I'd be happy to update that after committing this or do a follow on after committing this. It'd be great to unit test the DDLSemanticAnalyzer code and that patch adds about 20 tests. ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java <https://reviews.apache.org/r/16565/#comment59389> Thanks for cleaning this up.. - Brock Noland On Jan. 2, 2014, 7:05 a.m., Thejas Nair wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/16565/ > ----------------------------------------------------------- > > (Updated Jan. 2, 2014, 7:05 a.m.) > > > Review request for hive and Brock Noland. > > > Repository: hive-git > > > Description > ------- > > Grant privilege and revoke privilege statements need to be changed to remove > the requirement (but not the option) for the noise word TABLE. In the SQL > specification table is the assumed default for grant and revoke statements. > Today Hive’s syntax is GRANT action ON TABLE table TO grantee. It should be > GRANT action ON [TABLE] table TO grantee. > > Grant role and revoke role statements has been changed to remove the need for > keyword ROLE. > > Support for WITH ADMIN OPTION needs to be added to grant role and revoke role > statement syntax. > > HIVE-5923.3.patch > > > Diffs > ----- > > ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java 8d0b7a8 > ql/src/java/org/apache/hadoop/hive/ql/parse/DDLSemanticAnalyzer.java > a1b19a6 > ql/src/java/org/apache/hadoop/hive/ql/parse/HiveLexer.g 9354637 > ql/src/java/org/apache/hadoop/hive/ql/parse/HiveParser.g b34f53b > ql/src/java/org/apache/hadoop/hive/ql/parse/IdentifiersParser.g 39d53d3 > ql/src/java/org/apache/hadoop/hive/ql/plan/PrivilegeObjectDesc.java f0acb16 > ql/src/test/queries/clientpositive/authorization_2.q 4fc79b9 > ql/src/test/queries/clientpositive/authorization_role_grant1.q PRE-CREATION > ql/src/test/results/clientpositive/authorization_2.q.out 460240b > ql/src/test/results/clientpositive/authorization_role_grant1.q.out > PRE-CREATION > > Diff: https://reviews.apache.org/r/16565/diff/ > > > Testing > ------- > > > Thanks, > > Thejas Nair > >
