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.
Ashutosh Chauhan 2014-02-08, 02:39

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

(Updated Feb. 8, 2014, 2:39 a.m.)
Review request for hive.
Changes

Incorporated Thejas comments.
Bugs: HIVE-5930
    https://issues.apache.org/jira/browse/HIVE-5930
Repository: hive-git
Description

Implements set role and show current role functionality.
Diffs (updated)

  cli/src/java/org/apache/hadoop/hive/cli/CliDriver.java 06c3f6c
  ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java 32831fa
  ql/src/java/org/apache/hadoop/hive/ql/parse/DDLSemanticAnalyzer.java 9f15609
  ql/src/java/org/apache/hadoop/hive/ql/parse/HiveParser.g 7e69912
  ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzerFactory.java 2495c40
  ql/src/java/org/apache/hadoop/hive/ql/parse/authorization/HiveAuthorizationTaskFactory.java 1416c2e
  ql/src/java/org/apache/hadoop/hive/ql/parse/authorization/HiveAuthorizationTaskFactoryImpl.java e91258a
  ql/src/java/org/apache/hadoop/hive/ql/plan/RoleDDLDesc.java 77853c5
  ql/src/java/org/apache/hadoop/hive/ql/processors/CommandProcessorFactory.java 0ad2fde
  ql/src/java/org/apache/hadoop/hive/ql/processors/HiveCommand.java 280d94e
  ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAccessController.java 008efb1
  ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAuthorizer.java 632901e
  ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAuthorizerFactory.java c004105
  ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAuthorizerImpl.java 172746e
  ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLStdHiveAccessController.java 5c5d0e5
  ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLStdHiveAuthorizerFactory.java 7688bbf
  ql/src/test/org/apache/hadoop/hive/ql/processors/TestCommandProcessorFactory.java 732897f
  ql/src/test/queries/clientpositive/authorization_set_show_current_role.q PRE-CREATION
  ql/src/test/results/clientpositive/authorization_set_show_current_role.q.out PRE-CREATION
  service/src/java/org/apache/hive/service/cli/operation/ExecuteStatementOperation.java d00db1c

Diff: https://reviews.apache.org/r/17859/diff/
Testing

New test case is added.
Thanks,

Ashutosh Chauhan