Re: Review Request 16643: SQL std auth - support 'show all roles'

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16643/#review31700 --- Ship it! Ship It! - Thejas Nair On Jan. 13, 2014, 8:50 a.m., Nav

Re: Review Request 16643: SQL std auth - support 'show all roles'

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16643/ --- (Updated Jan. 13, 2014, 8:50 a.m.) Review request for hive. Changes ---

Re: Review Request 16643: SQL std auth - support 'show all roles'

> On Jan. 6, 2014, 7:49 a.m., Thejas Nair wrote: > > ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java, line 792 > > > > > > I think we should skip the "role name:" in the output, similar to the > > change you m

Re: Review Request 16643: SQL std auth - support 'show all roles'

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16643/ --- (Updated Jan. 13, 2014, 1:40 a.m.) Review request for hive. Changes ---

Re: Review Request 16643: SQL std auth - support 'show all roles'

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16643/#review31225 --- ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java

Review Request 16643: SQL std auth - support 'show all roles'

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