Alexey Serbin has posted comments on this change.

Change subject: monotime: remove granularity argument
......................................................................


Patch Set 1: Code-Review+1

(1 comment)

http://gerrit.cloudera.org:8080/#/c/3966/1//COMMIT_MSG
Commit Message:

Line 13: Let's remove it for now so that the API is a little simpler. If we ever
Having this clean-up is great!

As an alternative, consider using FINE as parameter-by-default for 
MonoTime::Now().

Or it contradicts with our current C++ code guideline?


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

Gerrit-MessageType: comment
Gerrit-Change-Id: If1d375d54e8598105f3ec833c37d96a382c1a8f7
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <d...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mpe...@apache.org>
Gerrit-HasComments: Yes

Reply via email to