Adar Dembo has posted comments on this change. Change subject: docs: Add documentation for background tasks ......................................................................
Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/6501/1/docs/background_tasks.adoc File docs/background_tasks.adoc: PS1, Line 53: maintain Maybe "maintain consistent"? PS1, Line 66: Kudu maintains a write-ahead log (WAL) per tablet Mention here that it's split up into discrete fixed-size segments, to add context for the next sentence. Line 73: Sometimes, it may be desirable to keep more WAL segments then the minimum While useful, most of this paragraph is somewhat out of scope of this doc. Line 81: `--log_max_segments_to_retain`. Not sure we should doc log_max_segments_to_retain as it's experimental. log_min_segments_to_retain is fair game though. PS1, Line 85: MVCC (multiversion concurrency control) Nit: reverse and place MVCC in parens. -- To view, visit http://gerrit.cloudera.org:8080/6501 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I58b14e481fad60397bbefc071887b7507e7806d0 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Mike Percy <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-HasComments: Yes
