Todd Lipcon has posted comments on this change.

Change subject: Various comment / doc improvements
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/3977/1/src/kudu/tablet/delta_compaction.cc
File src/kudu/tablet/delta_compaction.cc:

PS1, Line 182: Any other types of mutations in the compacted columns will
             :     //    now be either UNDO mutations or base data.
this sentence doesn't make much sense to me -- base data isn't a mutation, and 
not sure what "mutations in the compacted columns" really means. Maybe just 
remove it?


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I45d45d265652b0c462aec1bedbcb4b254a2a05da
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Mike Percy <mpe...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-HasComments: Yes

Reply via email to