Repository: incubator-sentry Updated Branches: refs/heads/master 7b17cef73 -> 7e3f287ca
SENTRY-368: Remove unused field in SentryPolicyServiceClient.java (Dapeng Sun via Sravya Tirukkovalur) Project: http://git-wip-us.apache.org/repos/asf/incubator-sentry/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-sentry/commit/7e3f287c Tree: http://git-wip-us.apache.org/repos/asf/incubator-sentry/tree/7e3f287c Diff: http://git-wip-us.apache.org/repos/asf/incubator-sentry/diff/7e3f287c Branch: refs/heads/master Commit: 7e3f287ca9413cb2dd0c4c76ea352fb18048f295 Parents: 7b17cef Author: Sravya Tirukkovalur <[email protected]> Authored: Mon Aug 4 13:37:23 2014 -0700 Committer: Sravya Tirukkovalur <[email protected]> Committed: Mon Aug 4 13:37:23 2014 -0700 ---------------------------------------------------------------------- .../provider/db/service/thrift/SentryPolicyServiceClient.java | 3 --- 1 file changed, 3 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-sentry/blob/7e3f287c/sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryPolicyServiceClient.java ---------------------------------------------------------------------- diff --git a/sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryPolicyServiceClient.java b/sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryPolicyServiceClient.java index 5b532ce..d4c5806 100644 --- a/sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryPolicyServiceClient.java +++ b/sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryPolicyServiceClient.java @@ -208,7 +208,6 @@ public class SentryPolicyServiceClient { /** * Gets sentry role objects for a given groupName using the Sentry service * @param requestorUserName : user on whose behalf the request is issued - * @param requestorUserGroupNames :groups the requesting user belongs to * @param groupName : groupName to look up ( if null returns all roles for all groups) * @return Set of thrift sentry role objects * @throws SentryUserException @@ -222,7 +221,6 @@ public class SentryPolicyServiceClient { request.setRequestorUserName(requestorUserName); request.setGroupName(groupName); TListSentryRolesResponse response; - Set<String> roles = new HashSet<String>(); try { response = client.list_sentry_roles_by_group(request); Status.throwIfNotOk(response.getStatus()); @@ -240,7 +238,6 @@ public class SentryPolicyServiceClient { /** * Gets sentry privilege objects for a given roleName using the Sentry service * @param requestorUserName : user on whose behalf the request is issued - * @param requestorUserGroupNames :groups the requesting user belongs to * @param roleName : roleName to look up * @param authorizable : authorizable Hierarchy (server->db->table etc) * @return Set of thrift sentry privilege objects
