You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kudu.apache.org by "David Ribeiro Alves (Code Review)" <ge...@ip-10-146-233-104.ec2.internal> on 2016/02/02 21:36:07 UTC

[kudu-CR] Change timestamp printing to ISO-8601 (with microseconds)

Hello Todd Lipcon,

I'd like you to do a code review.  Please visit

    http://gerrit.cloudera.org:8080/1995

to review the following change.

Change subject: Change timestamp printing to ISO-8601 (with microseconds)
......................................................................

Change timestamp printing to ISO-8601 (with microseconds)

Dan raised the concern in a recent review that we're not adhering to the
standard while printing timemstamps. The main point of this was to mimic
Impala's behavior but then we would also print 'GMT', which impala does
not, meaning we aren't really mimicking impala either.

This changes timestamp printing to adhere to ISO-8601, hopefully allowing
standard date parsers to more easily process the output, if needed.

Change-Id: I421e3595af2b21eee6ec22606b6046e470559ad4
---
M java/kudu-client/src/main/java/org/kududb/client/RowResult.java
M src/kudu/common/types-test.cc
M src/kudu/common/types.h
3 files changed, 7 insertions(+), 7 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/95/1995/1
-- 
To view, visit http://gerrit.cloudera.org:8080/1995
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I421e3595af2b21eee6ec22606b6046e470559ad4
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: David Ribeiro Alves <da...@cloudera.com>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>