You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@sentry.apache.org by Dapeng Sun <da...@intel.com> on 2016/02/01 02:25:59 UTC

Re: Review Request 42944: SENTRY-1027: Fix PMD error for unused field when enable Hive authz V2

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

(Updated 二月 1, 2016, 9:25 a.m.)


Review request for sentry.


Changes
-------

Update patch accroding the feedback.


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


Repository: sentry


Description
-------

Fix unused fields according PMD


Diffs (updated)
-----

  sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/DefaultSentryAccessController.java 9e72b78 

Diff: https://reviews.apache.org/r/42944/diff/


Testing
-------

Remove unused fields **grantorPrincipal at grantOrRevokePrivlegeOnRole** and **grantOption at grantOrRevokePrivlegeOnRole**


Thanks,

Dapeng Sun