Hello Marton Greber,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/19097
to look at the new patch set (#2).
Change subject: [WIP] KUDU-1945 Auto-Increment Column
......................................................................
[WIP] KUDU-1945 Auto-Increment Column
This patch adds a new column specification named auto_increment. 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 increment counter. This is a step towards
having tables with non unique primary keys or in case of a single
partitioned tables, a table wide unique key.
Upon receiving a write request the leader replica:
1. Fills in the replicate message with the auto increment counter
value for the first write op.
2. Populates the auto increment key into the rows being inserted
during the prepare phase.
3. Sends out the replicate message with the auto increment counter
and the original write request. The clients perform the same
set of steps to populate the auto increment column.
It also adds the c++ client side changes needed for basic writes
and reads of this column to perform end-to-end tests.
Next steps in the future revisions of this patch include more
testing of the functionality.
The feedback in this patch is needed on the implementation idea
on the server side.
Change-Id: I1dbde9095da78f6d1bd00adcc0a6e7dd63082bbc
---
M src/kudu/client/client-test.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/common/common.proto
M src/kudu/common/row_operations.cc
M src/kudu/common/row_operations.h
M src/kudu/common/schema.cc
M src/kudu/common/schema.h
M src/kudu/common/wire_protocol.cc
M src/kudu/consensus/consensus.proto
M src/kudu/tablet/CMakeLists.txt
M src/kudu/tablet/ops/write_op.cc
M src/kudu/tablet/tablet.cc
M src/kudu/tablet/tablet.h
A src/kudu/tablet/tablet_auto_increment-test.cc
M src/kudu/tablet/tablet_replica.cc
18 files changed, 325 insertions(+), 26 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/97/19097/2
--
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: 2
Gerrit-Owner: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Marton Greber <[email protected]>
