Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
Hive >> mail # dev >> Review Request 17859: Implements set role and show current role functionality.


Copy link to this message
-
Re: Review Request 17859: Implements set role and show current role functionality.

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

ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLStdHiveAccessController.java
<https://reviews.apache.org/r/17859/#comment63826>

    It's possible the exception doesn't have a cause and therefore we are eating the true exception.
    
    This should be throw new ...("msg here", e);

ql/src/test/org/apache/hadoop/hive/ql/processors/TestCommandProcessorFactory.java
<https://reviews.apache.org/r/17859/#comment63827>

    Why not add a new test for set role?
- Brock Noland
On Feb. 7, 2014, 10:22 p.m., Ashutosh Chauhan wrote: