Hello Mahesh Reddy, Marton Greber, Alexey Serbin, Kudu Jenkins, Wenzhe Zhou,

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

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

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

Change subject: KUDU-1945 Auto-Incrementing Column
......................................................................

KUDU-1945 Auto-Incrementing Column

This patch adds a new column specification named auto_incrementing.
These columns are populated on the server side with a monotonically
increasing counter. This counter is local to every tablet i.e. each
tablet has a separate auto incrementing counter. This is a step towards
having tables with non unique primary keys or in case of tables with
just one tablet, a table wide unique key.
Upon receiving a write request the leader replica:
1. Fills in the replicate message with the auto incrementing counter
   value for the first write op.
2. Populates the auto incrementing key into the rows being inserted
   during the prepare phase.
3. Sends out the replicate message with the auto incrementing counter
   and the original write request. The followers perform the same
   set of steps to populate the auto incrementing column.
It also adds the c++ client side changes needed for basic writes
and reads of this column to perform end-to-end tests.

Change-Id: I1dbde9095da78f6d1bd00adcc0a6e7dd63082bbc
---
M src/kudu/client/client-test.cc
M src/kudu/client/client-unittest.cc
M src/kudu/client/scan_configuration.cc
M src/kudu/client/schema-internal.h
M src/kudu/client/schema.cc
M src/kudu/client/schema.h
M src/kudu/client/session-internal.cc
M src/kudu/codegen/codegen-test.cc
M src/kudu/common/column_predicate-test.cc
M src/kudu/common/common.proto
M src/kudu/common/generic_iterators-test.cc
M src/kudu/common/partial_row-test.cc
M src/kudu/common/partition-test.cc
M src/kudu/common/row_operations.cc
M src/kudu/common/row_operations.h
M src/kudu/common/schema-test.cc
M src/kudu/common/schema.cc
M src/kudu/common/schema.h
M src/kudu/common/wire_protocol-test.cc
M src/kudu/common/wire_protocol.cc
M src/kudu/integration-tests/CMakeLists.txt
A src/kudu/integration-tests/auto_incrementing-itest.cc
M src/kudu/integration-tests/raft_consensus-itest-base.cc
M src/kudu/tablet/CMakeLists.txt
M src/kudu/tablet/all_types-scan-correctness-test.cc
M src/kudu/tablet/cfile_set-test.cc
M src/kudu/tablet/diskrowset-test.cc
M src/kudu/tablet/ops/write_op.cc
M src/kudu/tablet/tablet-decoder-eval-test.cc
M src/kudu/tablet/tablet-test-util.h
M src/kudu/tablet/tablet-test.cc
M src/kudu/tablet/tablet.cc
M src/kudu/tablet/tablet.h
A src/kudu/tablet/tablet_auto_incrementing-test.cc
M src/kudu/tablet/tablet_replica.cc
M src/kudu/tools/kudu-tool-test.cc
M src/kudu/tserver/tablet_server_authorization-test.cc
M src/kudu/tserver/tserver.proto
38 files changed, 661 insertions(+), 70 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/97/19097/10
--
To view, visit http://gerrit.cloudera.org:8080/19097
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I1dbde9095da78f6d1bd00adcc0a6e7dd63082bbc
Gerrit-Change-Number: 19097
Gerrit-PatchSet: 10
Gerrit-Owner: Abhishek Chennaka <[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: Marton Greber <[email protected]>
Gerrit-Reviewer: Wenzhe Zhou <[email protected]>

Reply via email to