Repository: sentry Updated Branches: refs/heads/master 3f32313d1 -> 25f025f3c
SENTRY-2267: Listing user privileges fails because roleName field is required on Thrift (Sergio Pena, reviewed by Na Li, Arjun Mishra) Project: http://git-wip-us.apache.org/repos/asf/sentry/repo Commit: http://git-wip-us.apache.org/repos/asf/sentry/commit/25f025f3 Tree: http://git-wip-us.apache.org/repos/asf/sentry/tree/25f025f3 Diff: http://git-wip-us.apache.org/repos/asf/sentry/diff/25f025f3 Branch: refs/heads/master Commit: 25f025f3ca89b1e4b4ce7adee8477b4d130214d7 Parents: 3f32313 Author: Sergio Pena <[email protected]> Authored: Wed Jun 13 09:27:06 2018 -0500 Committer: Sergio Pena <[email protected]> Committed: Wed Jun 13 09:27:06 2018 -0500 ---------------------------------------------------------------------- .../api/service/thrift/SentryPolicyServiceClientDefaultImpl.java | 2 ++ 1 file changed, 2 insertions(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/sentry/blob/25f025f3/sentry-service/sentry-service-api/src/main/java/org/apache/sentry/api/service/thrift/SentryPolicyServiceClientDefaultImpl.java ---------------------------------------------------------------------- diff --git a/sentry-service/sentry-service-api/src/main/java/org/apache/sentry/api/service/thrift/SentryPolicyServiceClientDefaultImpl.java b/sentry-service/sentry-service-api/src/main/java/org/apache/sentry/api/service/thrift/SentryPolicyServiceClientDefaultImpl.java index f0f08ea..45dce0e 100644 --- a/sentry-service/sentry-service-api/src/main/java/org/apache/sentry/api/service/thrift/SentryPolicyServiceClientDefaultImpl.java +++ b/sentry-service/sentry-service-api/src/main/java/org/apache/sentry/api/service/thrift/SentryPolicyServiceClientDefaultImpl.java @@ -267,6 +267,8 @@ public class SentryPolicyServiceClientDefaultImpl implements SentryPolicyService TListSentryPrivilegesRequest request = new TListSentryPrivilegesRequest(); request.setProtocol_version(ThriftConstants.TSENTRY_SERVICE_VERSION_CURRENT); request.setRequestorUserName(requestorUserName); + // TODO: Remove setRoleName() once the required field is removed + request.setRoleName(""); // roleName is unused by it is required by Thrift request.setEntityName(userName); if (authorizable != null && !authorizable.isEmpty()) { TSentryAuthorizable tSentryAuthorizable = setupSentryAuthorizable(authorizable);
