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....@cloudera.com> Authored: Wed Sep 26 11:39:50 2018 -0700 Committer: 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 {