Re: Review Request 16034: Add explain authorize for checking privileges

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16034/#review43899 --- Ship it! Ship It! - Ashutosh Chauhan On May 24, 2014, 8:45 a.m.,

Re: Review Request 16034: Add explain authorize for checking privileges

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16034/ --- (Updated May 24, 2014, 8:45 a.m.) Review request for hive. Changes --- A

Re: Review Request 16034: Add explain authorize for checking privileges

> On May 24, 2014, 12:51 a.m., Ashutosh Chauhan wrote: > > ql/src/java/org/apache/hadoop/hive/ql/exec/ExplainTask.java, line 325 > > > > > > This if condition can never be false, right ? Simple explain needs full auth

Re: Review Request 16034: Add explain authorize for checking privileges

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16034/#review43887 --- ql/src/java/org/apache/hadoop/hive/ql/Driver.java

Re: Review Request 16034: Add explain authorize for checking privileges

--- 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 ---

Re: Review Request 16034: Add explain authorize for checking privileges

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16034/ --- (Updated Feb. 7, 2014, 2:31 a.m.) Review request for hive. Changes --- R

Re: Review Request 16034: Add explain authorize for checking privileges

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16034/ --- (Updated Dec. 10, 2013, 12:57 a.m.) Review request for hive. Bugs: HIVE-5961

Review Request 16034: Add explain authorize for checking privileges

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16034/ --- Review request for hive. Bugs: HIVE-5961 https://issues.apache.org/jira/bro