sentry-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sp...@apache.org
Subject sentry git commit: SENTRY-2267: Listing user privileges fails because roleName field is required on Thrift (Sergio Pena, reviewed by Na Li, Arjun Mishra)
Date Wed, 13 Jun 2018 14:27:31 GMT
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 <sergio.pena@cloudera.com>
Authored: Wed Jun 13 09:27:06 2018 -0500
Committer: Sergio Pena <sergio.pena@cloudera.com>
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);


Mime
View raw message