Hello Tidy Bot, Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/8533 to look at the new patch set (#3). Change subject: Add initial internal INT128/__int128 support ...................................................................... Add initial internal INT128/__int128 support This patch adds internal support for INT128/__int128 in Kudu. This preliminary functionality is required to support the DECIMAL type in KUDU-721. Follow up patches will be added to review and discus making the INT128 column type publicly exposed. Change-Id: I36458a54dfdd28be03f80d83688c0d658944e8e1 --- M src/kudu/cfile/bshuf_block.h M src/kudu/cfile/cfile-test-base.h M src/kudu/cfile/encoding-test.cc M src/kudu/cfile/type_encodings.cc M src/kudu/common/common.proto M src/kudu/common/types.cc M src/kudu/common/types.h M src/kudu/gutil/CMakeLists.txt M src/kudu/gutil/endian.h M src/kudu/gutil/integral_types.h M src/kudu/gutil/mathlimits.cc M src/kudu/gutil/mathlimits.h A src/kudu/gutil/strings/numbers-test.cc M src/kudu/gutil/strings/numbers.cc M src/kudu/gutil/strings/numbers.h M src/kudu/gutil/strings/substitute.h M src/kudu/gutil/type_traits.h M src/kudu/util/CMakeLists.txt A src/kudu/util/int128-test.cc A src/kudu/util/int128.cc A src/kudu/util/int128.h 21 files changed, 294 insertions(+), 0 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/33/8533/3 -- To view, visit http://gerrit.cloudera.org:8080/8533 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I36458a54dfdd28be03f80d83688c0d658944e8e1 Gerrit-Change-Number: 8533 Gerrit-PatchSet: 3 Gerrit-Owner: Grant Henke <granthe...@gmail.com> Gerrit-Reviewer: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: Grant Henke <granthe...@gmail.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot