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

Review request for sentry.


Bugs: SENTRY-1979
    https://issues.apache.org/jira/browse/SENTRY-1979


Repository: sentry


Description
-------

This task is to consolidate the code for converting Hive privileges objects to 
Strings. We have some duplicate code that is used by the shell that could be 
removed.


Diffs
-----

  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/tools/command/hive/CommandUtil.java
 51ee9efc 
  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/tools/command/hive/GrantPrivilegeToRoleCmd.java
 e3d06a97 
  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/tools/command/hive/ListPrivilegesCmd.java
 5f3e9fbc 
  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/tools/command/hive/RevokePrivilegeFromRoleCmd.java
 fe6aca5d 
  
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/tools/TestSentryShellHive.java
 81059c5f 


Diff: https://reviews.apache.org/r/63026/diff/1/


Testing
-------

Tested the Hive Shell.


Thanks,

Colm O hEigeartaigh

Reply via email to