Re: Review Request 30575: HIVE-9350 : Add ability for HiveAuthorizer implementations to filter out results of 'show tables', 'show databases'

2015-02-13 Thread Thejas Nair
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30575/ --- (Updated Feb. 13, 2015, 7 p.m.) Review request for hive and Jason Dere.

Review Request 30575: HIVE-9350 : Add ability for HiveAuthorizer implementations to filter out results of 'show tables', 'show databases'

2015-02-03 Thread Thejas Nair
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30575/ --- Review request for hive and Jason Dere. Bugs: HIVE-9350

Re: Review Request 30575: HIVE-9350 : Add ability for HiveAuthorizer implementations to filter out results of 'show tables', 'show databases'

2015-02-03 Thread Thejas Nair
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30575/ --- (Updated Feb. 3, 2015, 7:27 p.m.) Review request for hive and Jason Dere.

Re: Review Request 30575: HIVE-9350 : Add ability for HiveAuthorizer implementations to filter out results of 'show tables', 'show databases'

2015-02-03 Thread Jason Dere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30575/#review70871 --- ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java

Re: Review Request 30575: HIVE-9350 : Add ability for HiveAuthorizer implementations to filter out results of 'show tables', 'show databases'

2015-02-03 Thread Thejas Nair
On Feb. 4, 2015, 12:15 a.m., Jason Dere wrote: ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java, line 710 https://reviews.apache.org/r/30575/diff/2/?file=846469#file846469line710 Perhaps mention in a comment that this should only be called for V2 authorization, or

Re: Review Request 30575: HIVE-9350 : Add ability for HiveAuthorizer implementations to filter out results of 'show tables', 'show databases'

2015-02-03 Thread Thejas Nair
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30575/ --- (Updated Feb. 4, 2015, 1:30 a.m.) Review request for hive and Jason Dere.