hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jonathan Natkins" <na...@cloudera.com>
Subject Re: Review Request: HIVE-1928. GRANT/REVOKE should handle privileges as tokens, not identifiers
Date Thu, 17 Feb 2011 00:17:14 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/427/
-----------------------------------------------------------

(Updated 2011-02-16 16:17:14.544784)


Review request for hive.


Changes
-------

Updating with an additional test change


Summary
-------

Review request for HIVE-1928.


This addresses bug HIVE-1928.
    https://issues.apache.org/jira/browse/HIVE-1928


Diffs (updated)
-----

  ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java 6fea990 
  ql/src/java/org/apache/hadoop/hive/ql/parse/DDLSemanticAnalyzer.java 9d9cea1 
  ql/src/java/org/apache/hadoop/hive/ql/parse/Hive.g c5574b0 
  ql/src/java/org/apache/hadoop/hive/ql/security/authorization/DefaultHiveAuthorizationProvider.java
89af39d 
  ql/src/java/org/apache/hadoop/hive/ql/security/authorization/Privilege.java 1449091 
  ql/src/java/org/apache/hadoop/hive/ql/security/authorization/PrivilegeRegistry.java 7cfc0e5

  ql/src/test/queries/clientnegative/authorization_fail_1.q 063ef52 
  ql/src/test/queries/clientnegative/authorization_fail_3.q 1d45ad6 
  ql/src/test/queries/clientnegative/authorization_fail_4.q b58288b 
  ql/src/test/queries/clientnegative/authorization_fail_5.q 5be2965 
  ql/src/test/queries/clientnegative/authorization_fail_7.q ef80525 
  ql/src/test/queries/clientnegative/authorization_part.q 2d5efe3 
  ql/src/test/queries/clientpositive/authorization_1.q 0a26c2e 
  ql/src/test/queries/clientpositive/authorization_2.q 733cf5e 
  ql/src/test/queries/clientpositive/authorization_3.q 577886a 
  ql/src/test/queries/clientpositive/authorization_4.q d679d78 
  ql/src/test/queries/clientpositive/authorization_5.q df809f0 
  ql/src/test/queries/clientpositive/keyword_1.q c7fc640 
  ql/src/test/results/clientnegative/authorization_fail_1.q.out ad3b956 
  ql/src/test/results/clientnegative/authorization_fail_3.q.out 37f819b 
  ql/src/test/results/clientnegative/authorization_fail_4.q.out 3136b53 
  ql/src/test/results/clientnegative/authorization_fail_5.q.out fb599de 
  ql/src/test/results/clientnegative/authorization_fail_7.q.out 898e6f5 
  ql/src/test/results/clientnegative/authorization_part.q.out 028d3e3 
  ql/src/test/results/clientpositive/authorization_1.q.out 319d7ab 
  ql/src/test/results/clientpositive/authorization_2.q.out 5e947e4 
  ql/src/test/results/clientpositive/authorization_3.q.out 1006d52 
  ql/src/test/results/clientpositive/authorization_4.q.out fa3fa9d 
  ql/src/test/results/clientpositive/authorization_5.q.out 7578981 
  ql/src/test/results/clientpositive/keyword_1.q.out 4a0a194 

Diff: https://reviews.apache.org/r/427/diff


Testing
-------


Thanks,

Jonathan


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message