Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13022 )
Change subject: WIP [docs] updated master-related scenarios w.r.t. CM ...................................................................... Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/13022/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/13022/1//COMMIT_MSG@9 PS1, Line 9: WIP: Do we need to mention CM in the upstream doc at all? : Maybe, we should remove all CM-related references : from this doc? Maybe. But this patch is a strict improvement over what came before, so I feel like I can approve of it without wading into the larger conversation about whether CM should be mentioned at all. http://gerrit.cloudera.org:8080/#/c/13022/1/docs/administration.adoc File docs/administration.adoc: http://gerrit.cloudera.org:8080/#/c/13022/1/docs/administration.adoc@754 PS1, Line 754: run-time flag Nit: I think "configuration parameter", "parameter", or "flag" are better options than "run-time flag" here and below. -- To view, visit http://gerrit.cloudera.org:8080/13022 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I96bf3ec0de07a46151f947a18b9466da5f05adb3 Gerrit-Change-Number: 13022 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Attila Bukor <abu...@apache.org> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com> Gerrit-Comment-Date: Tue, 16 Apr 2019 22:42:50 +0000 Gerrit-HasComments: Yes