David Ribeiro Alves has submitted this change and it was merged. 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 Reviewed-on: http://gerrit.cloudera.org:8080/965 Tested-by: Kudu Jenkins Reviewed-by: Dan Burkert <[email protected]> --- 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(-) Approvals: Dan Burkert: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/965 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I97eb088a4d0ef082ab9c7e3cca40e809b0f29934 Gerrit-PatchSet: 10 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]>
