Hello Michael Ho, Dan Hecht, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/8305 to look at the new patch set (#2). Change subject: IMPALA-5599: Clean up references to TimestampValue in be/src. ...................................................................... IMPALA-5599: Clean up references to TimestampValue in be/src. This is a follow-on commit to d53f43b4. In this patch we replace all inappropriate usage of TimestampValue in be/src/service and be/src/statetore with simpler data types for Unix timestamps, and where conversions to strings are needed, we use the interfaces introduced by the previous commit. Change-Id: I642a1d713597826bb7c15cd2ecb6638cb813a02c --- M be/src/service/client-request-state.cc M be/src/service/client-request-state.h M be/src/service/impala-http-handler.cc M be/src/service/impala-server.cc M be/src/service/impala-server.h M be/src/statestore/statestore-subscriber.cc 6 files changed, 29 insertions(+), 42 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/05/8305/2 -- To view, visit http://gerrit.cloudera.org:8080/8305 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I642a1d713597826bb7c15cd2ecb6638cb813a02c Gerrit-Change-Number: 8305 Gerrit-PatchSet: 2 Gerrit-Owner: Zoram Thanga <zo...@cloudera.com> Gerrit-Reviewer: Dan Hecht <dhe...@cloudera.com> Gerrit-Reviewer: Michael Ho <k...@cloudera.com> Gerrit-Reviewer: Zoram Thanga <zo...@cloudera.com>