Will Berkeley has uploaded a new change for review. http://gerrit.cloudera.org:8080/5104
Change subject: [WIP] KUDU-721 Support for Decimal type: Part 1 (C++) ...................................................................... [WIP] KUDU-721 Support for Decimal type: Part 1 (C++) This patch introduces a Decimal type to Kudu. It's based on the column attributes, which are bits of information attached to a ColumnSchema. For a Decimal column, the attributes are the precision and scale of the Decimal column. Decimal attributes determine 1. The server-side storage/physical type of the column, either INT32 or INT64. This should be invisible to clients. 2. How clients should interpret the values of a decimal column. No other types use attributes at this time. The design of the Decimal column is based on Impala's decimal type. The maximum precision is 18, the largest precision whose maximum value can be stored in a 64-bit integer. This patch contains only server-side and C++ client Decimal support. Java support will follow. Change-Id: I4f7829cdad883f5c124a437327062b2b45b6841b --- M src/kudu/client/CMakeLists.txt M src/kudu/client/client-unittest.cc 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-internal.h M src/kudu/client/schema.cc M src/kudu/client/schema.h M src/kudu/client/value.cc M src/kudu/client/value.h M src/kudu/common/CMakeLists.txt M src/kudu/common/common.proto A src/kudu/common/decimal_value-test.cc A src/kudu/common/decimal_value.cc A src/kudu/common/decimal_value.h M src/kudu/common/partial_row.cc M src/kudu/common/partial_row.h M src/kudu/common/schema-test.cc M src/kudu/common/schema.cc M src/kudu/common/schema.h M src/kudu/common/types.cc M src/kudu/common/types.h M src/kudu/common/wire_protocol.cc M src/kudu/consensus/log-test.cc M src/kudu/integration-tests/all_types-itest.cc M src/kudu/master/catalog_manager.cc M src/kudu/server/webui_util.cc M src/kudu/tablet/cfile_set-test.cc M src/kudu/tablet/multi_column_writer.cc M src/kudu/tablet/tablet-decoder-eval-test.cc 30 files changed, 1,156 insertions(+), 53 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/04/5104/1 -- To view, visit http://gerrit.cloudera.org:8080/5104 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I4f7829cdad883f5c124a437327062b2b45b6841b Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Will Berkeley <[email protected]>
