Todd Lipcon has posted comments on this change.

Change subject: cfile_writer: inline written_size() method
......................................................................


Patch Set 2:

Didn't get a chance to benchmark this change in isolation, but since it's a 
simple getter, it's more stylistic to inline it in the header anyway

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I40b140b283899ba8ce05e08f01e28d943c2ab6af
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: David Ribeiro Alves <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: No

Reply via email to