Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5819 to look at the new patch set (#5). Change subject: [timestamp] Add a new TIMESTAMP_NANOS type ...................................................................... [timestamp] Add a new TIMESTAMP_NANOS type This adds a new type to Kudu and adds the ability to use it in table primary keys. The new type is similar to Impala's TIMESTAMP type and takes the form of the new TimestampValue class introduced in a previous patch. This also adds tests for the new type to all_types-itest both as a regular column and as part of the primary key. Change-Id: I63271adf6b87f048e30bf2d6d04c758884752afc --- M src/kudu/cfile/type_encodings.cc M src/kudu/client/scan_batch.cc M src/kudu/client/scan_batch.h M src/kudu/client/schema.cc M src/kudu/client/schema.h M src/kudu/client/symbols.map M src/kudu/client/value-internal.h M src/kudu/client/value.cc M src/kudu/client/value.h M src/kudu/common/common.proto M src/kudu/common/encoded_key-test.cc M src/kudu/common/key_encoder.cc M src/kudu/common/key_encoder.h M src/kudu/common/key_util.cc M src/kudu/common/partial_row.cc M src/kudu/common/partial_row.h M src/kudu/common/partition.cc M src/kudu/common/types.cc M src/kudu/common/types.h M src/kudu/integration-tests/all_types-itest.cc 20 files changed, 467 insertions(+), 30 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/19/5819/5 -- To view, visit http://gerrit.cloudera.org:8080/5819 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I63271adf6b87f048e30bf2d6d04c758884752afc Gerrit-PatchSet: 5 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: David Ribeiro Alves <dral...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: David Ribeiro Alves <dral...@apache.org> Gerrit-Reviewer: Jean-Daniel Cryans <jdcry...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <t...@apache.org>