Andrew Wong has posted comments on this change.

Change subject: docs: update disk failure recovery notes
......................................................................


Patch Set 1:

(3 comments)

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

Line 597: empty all of the server's existing directories. For example, if a 
tablet server
> do you think a WARNING bar is appropriate here? I wonder if someone would f
Done


PS1, Line 608: along with any newly-added data
             : directories
> you mean 'and new data directories have been created with appropriate permi
Done


PS1, Line 630: suicide_on_eio=false`. When set, tablets with data on a failed 
disk
             : will not be opened and will be re-replicated as necessary
> given that we are currently defaulting to spreading tablets across all disk
Fair point about striping. Made it less of a point.

Also changed to --crash_on_eio; hopefully less controversial and made this 
patch dependent on the separate patch.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I732286d0f56f7a15705ad544fc7dfc426287714e
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: Yes

Reply via email to