zhen.zhang has posted comments on this change.

Change subject: docs: add notes for multi master migration
......................................................................


Patch Set 1:

(2 comments)

I just installed "Asciidoctor.js Live Preview", it's very useful, thanks :)

http://gerrit.cloudera.org:8080/#/c/5728/1/docs/administration.adoc
File docs/administration.adoc:

Line 292: NOTE:If the existing master's `fs_data_dirs` parameter is configured, 
you should also add it as a parameter of this format tool.
> Just so I understand, you've configured your _master_ with multiple data di
We configured to write wal in ssd, and write data in hdd, so wal path and data 
path are not the same. And yes we set master's fs_data_dirs with multiple data 
directories, because master and ts use the same configuration template... I'll 
try to fix this when building new cluster in the future.


Line 323: 
> Nit: reformat this new paragraph so that it adheres to the line  wrapping g
Sorry I don't get what you mean, as the length of this paragraph is not beyond 
the dashed red vertical line, and I tries to the the length same with other 
paragraphs.


-- 
To view, visit http://gerrit.cloudera.org:8080/5728
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I7e469bc23b650f3b99c785311acc665ac20f33ea
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: zhen.zhang <zhangz...@xiaomi.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: zhen.zhang <zhangz...@xiaomi.com>
Gerrit-HasComments: Yes

Reply via email to