sentry-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ha...@apache.org
Subject [1/2] sentry git commit: SENTRY-2417: LocalGroupMappingService class docs do not accurately reflect required INI format (Dan Burkert reviewed by Na Li, Sergio Pena)
Date Wed, 26 Sep 2018 18:58:34 GMT
Repository: sentry
Updated Branches:
  refs/heads/master c7c5df286 -> 831594907


SENTRY-2417: LocalGroupMappingService class docs do not accurately reflect required INI format
(Dan Burkert reviewed by Na Li, Sergio Pena)


Project: http://git-wip-us.apache.org/repos/asf/sentry/repo
Commit: http://git-wip-us.apache.org/repos/asf/sentry/commit/83159490
Tree: http://git-wip-us.apache.org/repos/asf/sentry/tree/83159490
Diff: http://git-wip-us.apache.org/repos/asf/sentry/diff/83159490

Branch: refs/heads/master
Commit: 8315949073ebca169ef18791a3e750cbfdfbb252
Parents: e90ab35
Author: Hao Hao <hao.hao@cloudera.com>
Authored: Wed Sep 26 11:39:50 2018 -0700
Committer: Hao Hao <hao.hao@cloudera.com>
Committed: Wed Sep 26 11:54:08 2018 -0700

----------------------------------------------------------------------
 .../provider/file/LocalGroupMappingService.java | 26 +++++++++-----------
 1 file changed, 11 insertions(+), 15 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/sentry/blob/83159490/sentry-provider/sentry-provider-file/src/main/java/org/apache/sentry/provider/file/LocalGroupMappingService.java
----------------------------------------------------------------------
diff --git a/sentry-provider/sentry-provider-file/src/main/java/org/apache/sentry/provider/file/LocalGroupMappingService.java
b/sentry-provider/sentry-provider-file/src/main/java/org/apache/sentry/provider/file/LocalGroupMappingService.java
index 4430ce7..c138072 100644
--- a/sentry-provider/sentry-provider-file/src/main/java/org/apache/sentry/provider/file/LocalGroupMappingService.java
+++ b/sentry-provider/sentry-provider-file/src/main/java/org/apache/sentry/provider/file/LocalGroupMappingService.java
@@ -41,24 +41,20 @@ import com.google.common.base.Strings;
 import com.google.common.collect.Sets;
 
 /**
- * Mapping users to groups
- * parse the ini file with section [users] that contains the user names.
- * For each user in that list, there's section that contains the group
- * name for that user If there's no user section or no group section for
- * one of users, then just print a warning and continue.
- * Example -
- * [users]
- * usr1
- * usr2
+ * User to group mapping service for testing.
  *
- * [[usr1]
- * group1
- * group11
+ * The user to group mapping is configured via an INI file, located via the
+ * <pre>sentry.store.group.mapping.resource</pre> configuration.
  *
- * [usr2]
- * group21
- * group22
+ * Each user in the configuration is associated with zero or more
+ * comma-separated groups, for example:
  *
+ * <pre>
+ * [users]
+ * usr1=grp1
+ * usr2=grp1,grp2
+ * usr3=""
+ * </pre>
  */
 public class LocalGroupMappingService implements GroupMappingService {
 


Mime
View raw message