Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13780 )
Change subject: [docs] Add admin docs for backup and restore ...................................................................... Patch Set 1: (10 comments) http://gerrit.cloudera.org:8080/#/c/13780/1/docs/administration.adoc File docs/administration.adoc: http://gerrit.cloudera.org:8080/#/c/13780/1/docs/administration.adoc@303 PS1, Line 303: Spark compatible nit: Spark-compatible, here and below http://gerrit.cloudera.org:8080/#/c/13780/1/docs/administration.adoc@310 PS1, Line 310: KuduBackup nit: backticks http://gerrit.cloudera.org:8080/#/c/13780/1/docs/administration.adoc@323 PS1, Line 323: les th nit: add a comma http://gerrit.cloudera.org:8080/#/c/13780/1/docs/administration.adoc@342 PS1, Line 342: Job nit: lowercase? http://gerrit.cloudera.org:8080/#/c/13780/1/docs/administration.adoc@344 PS1, Line 344: KuduRestore nit: backticks http://gerrit.cloudera.org:8080/#/c/13780/1/docs/administration.adoc@371 PS1, Line 371: hadoop classpath Is this an actual command? If not, maybe this should be en environment variable? Or surrounded by <>s? http://gerrit.cloudera.org:8080/#/c/13780/1/docs/administration.adoc@396 PS1, Line 396: Currently the `toMs` time for the job. Not sure I understand this http://gerrit.cloudera.org:8080/#/c/13780/1/docs/administration.adoc@398 PS1, Line 398: linking Might be worth adding a bit of description somewhere for what linking means in the context of backup and restore? http://gerrit.cloudera.org:8080/#/c/13780/1/docs/administration.adoc@424 PS1, Line 424: General lowercase http://gerrit.cloudera.org:8080/#/c/13780/1/docs/administration.adoc@426 PS1, Line 426: Sparks Spark's -- To view, visit http://gerrit.cloudera.org:8080/13780 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I75f92d3f10fd5d970099e933d8de2d7662e03398 Gerrit-Change-Number: 13780 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Will Berkeley <[email protected]> Gerrit-Comment-Date: Tue, 02 Jul 2019 21:53:12 +0000 Gerrit-HasComments: Yes
