Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/965
to look at the new patch set (#8).
Change subject: KUDU-980 - Fix timestamp printing in c++
......................................................................
KUDU-980 - Fix timestamp printing in c++
The current time printing function, from gutil, had two problems:
- It only printed down to seconds.
- If it was passed 0 seconds, as it might in tests at least, it would
ignore the argument and return the current time.
This makes our time printing like Impala's (though with less precision).
Specifically this makes us print time in the format:
"YYYY-MM-DD HH:MM:SS.ssssss GMT"
Change-Id: I97eb088a4d0ef082ab9c7e3cca40e809b0f29934
---
M src/kudu/common/CMakeLists.txt
A src/kudu/common/types-test.cc
M src/kudu/common/types.h
3 files changed, 87 insertions(+), 7 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/65/965/8
--
To view, visit http://gerrit.cloudera.org:8080/965
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I97eb088a4d0ef082ab9c7e3cca40e809b0f29934
Gerrit-PatchSet: 8
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: David Ribeiro Alves <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: David Ribeiro Alves <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Wes McKinney <[email protected]>
