Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/13704 )
Change subject: [docs] Add docs for extra configuration properties ...................................................................... Patch Set 3: (3 comments) Thanks for comments, it looks much better now. :D http://gerrit.cloudera.org:8080/#/c/13704/1/docs/configuration.adoc File docs/configuration.adoc: http://gerrit.cloudera.org:8080/#/c/13704/1/docs/configuration.adoc@143 PS1, Line 143: Kudu allows certain configurations to be set per table. To configure the behavior of a Kudu table, : you can set th > nit: maybe add a little context about table-level configurations. Something Done http://gerrit.cloudera.org:8080/#/c/13704/1/docs/configuration.adoc@146 PS1, Line 146: > nit: maybe "Configurable Properties"? Done http://gerrit.cloudera.org:8080/#/c/13704/1/docs/configuration.adoc@150 PS1, Line 150: > I see. I agree isn't the most future proof thing ever. How about let's remo Done -- To view, visit http://gerrit.cloudera.org:8080/13704 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I7fe61efeafa9bd9d4cf62a378b6c4c7039bbcf6b Gerrit-Change-Number: 13704 Gerrit-PatchSet: 3 Gerrit-Owner: Yao Xu <oclarms....@gmail.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yao Xu <oclarms....@gmail.com> Gerrit-Comment-Date: Tue, 25 Jun 2019 04:13:46 +0000 Gerrit-HasComments: Yes