Hello Tidy Bot, Kudu Jenkins,

I'd like you to reexamine a change. Please visit

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

to look at the new patch set (#2).

Change subject: Add public INT128 column type support
......................................................................

Add public INT128 column type support

This patch adds basic support to the c++ client to
create, read, and write tables with INT128 columns.

128 bit integers are useful for many applications
such as UUIDs, IPv6 addresses, MD5 hashes, etc.
Given the various use cases we should consider
exposing the INT128 data type publicly to the clients.

The major risk with doing this is that downstream
integrations, such as Impala, Spark, Flume, etc,
may not be able to read INT128 columns without
special handling.

Change-Id: Icdfbe5a7d27c79e195db96f7dc5d71c8978520bf
---
M src/kudu/client/CMakeLists.txt
M src/kudu/client/predicate-test.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/value-internal.h
M src/kudu/client/value.cc
M src/kudu/client/value.h
M src/kudu/common/column_predicate-test.cc
M src/kudu/common/column_predicate.cc
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/row.h
M src/kudu/integration-tests/all_types-itest.cc
M src/kudu/tools/data_gen_util.cc
M src/kudu/tools/kudu-tool-test.cc
M src/kudu/tools/tool_action_perf.cc
M src/kudu/util/int128-test.cc
M src/kudu/util/int128.h
25 files changed, 160 insertions(+), 31 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/53/8553/2
--
To view, visit http://gerrit.cloudera.org:8080/8553
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Icdfbe5a7d27c79e195db96f7dc5d71c8978520bf
Gerrit-Change-Number: 8553
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot

Reply via email to