Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13902 )
Change subject: docs: add the required config for Sentry ...................................................................... Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/13902/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/13902/1//COMMIT_MSG@7 PS1, Line 7: docs: add the required config for Sentry > Could you add a note that this is for upstream only? Maybe something like, "Thirdparty cluster management software like Cloudera Manager may configure these automatically. If not using such software, it's necessary to set these configurations manually." http://gerrit.cloudera.org:8080/#/c/13902/1/docs/security.adoc File docs/security.adoc: http://gerrit.cloudera.org:8080/#/c/13902/1/docs/security.adoc@306 PS1, Line 306: ---sentry.service.allow.connect=kudu : ---sentry.service.admin.group=kudu Aren't these defined as XML configs? Like those defined here: https://kudu.apache.org/docs/hive_metastore.html#_enabling_the_hive_metastore_integration -- To view, visit http://gerrit.cloudera.org:8080/13902 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: If208579405c11fc25aefbedd3f63ac21afecbddf Gerrit-Change-Number: 13902 Gerrit-PatchSet: 1 Gerrit-Owner: Hao Hao <[email protected]> Gerrit-Reviewer: Alex Rodoni <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Tue, 23 Jul 2019 22:38:38 +0000 Gerrit-HasComments: Yes
