Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/18835 )
Change subject: KUDU-3353 [schema] Add an immutable attribute on column schema (part 2) ...................................................................... KUDU-3353 [schema] Add an immutable attribute on column schema (part 2) This is a follow-up to b6eedb224f715ad86378a92d25f09c2084b0e2b7. This patch includes the C++ client-side changes of the "new column attribute IMMUTABLE" feature, including: 1. Adds a new KuduColumnSpec::Immutable() method to add IMMUTABLE attribute to a column, and adds a new KuduColumnSpec::Mutable() method to remove IMMUTABLE attribute from a column. 2. Adds a new KuduColumnSchema::is_immutable() to check if the attribute is set for a column schema. 3. Adds a new KuduUpsertIgnore operation in the client API: use the newly added KuduTable::NewUpsertIgnore() to create a new instance of such operation. 4. Adds unit tests to cover the newly introduced functionality. 5. Small refactoring on the server side. Change-Id: Id301e313eedd9cc3a494d6b4f3c14fbabe63b436 Reviewed-on: http://gerrit.cloudera.org:8080/18835 Tested-by: Yingchun Lai <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> --- M src/kudu/client/client-internal.cc M src/kudu/client/client-internal.h M src/kudu/client/client-test.cc M src/kudu/client/client-unittest.cc M src/kudu/client/client.cc M src/kudu/client/client.h M src/kudu/client/schema-internal.h M src/kudu/client/schema.cc M src/kudu/client/schema.h M src/kudu/client/table_alterer-internal.cc M src/kudu/client/write_op.cc M src/kudu/client/write_op.h M src/kudu/common/row_operations.cc M src/kudu/integration-tests/alter_table-test.cc M src/kudu/master/master.proto M src/kudu/master/master_service.cc M src/kudu/tablet/ops/write_op.cc M src/kudu/tablet/row_op.cc M src/kudu/tablet/row_op.h M src/kudu/tablet/tablet.cc M src/kudu/tablet/tablet_metrics.cc 21 files changed, 725 insertions(+), 112 deletions(-) Approvals: Yingchun Lai: Verified Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/18835 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Id301e313eedd9cc3a494d6b4f3c14fbabe63b436 Gerrit-Change-Number: 18835 Gerrit-PatchSet: 17 Gerrit-Owner: Yingchun Lai <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mahesh Reddy <[email protected]> Gerrit-Reviewer: Yifan Zhang <[email protected]> Gerrit-Reviewer: Yingchun Lai <[email protected]>
