Hello Alexey Serbin, Kudu Jenkins,

I'd like you to do a code review. Please visit

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

to review the following change.


Change subject: KUDU-1945 Backup/restore for tables with auto incrementing 
columns
......................................................................

KUDU-1945 Backup/restore for tables with auto incrementing columns

This patch adds the ability to backup and restore tables containing
auto-incrementing columns. A simple test case is also added to test
the functionaility.

Without this patch any attempt to
Backup a table with auto-incrementing column will fail with an error
message "auto_incrementing_id is a reserved column name" from
schema.cc.
Restoring a table with auto-incrementing column will result in the
same behavior as with the backup.

Change-Id: I929d54d22c1c938ee67fdda9f4c2bb68c028b4ec
Reviewed-on: http://gerrit.cloudera.org:8080/20084
Reviewed-by: Alexey Serbin <ale...@apache.org>
Tested-by: Kudu Jenkins
(cherry picked from commit 1bdb0105b1ae64c167f32b67ee9da03825605d9c)
---
M java/kudu-backup-common/src/main/protobuf/backup.proto
M 
java/kudu-backup-common/src/main/scala/org/apache/kudu/backup/TableMetadata.scala
M java/kudu-backup/src/test/scala/org/apache/kudu/backup/TestKuduBackup.scala
M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/KuduTestSuite.scala
M src/kudu/common/schema.cc
5 files changed, 113 insertions(+), 28 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/29/20229/1
--
To view, visit http://gerrit.cloudera.org:8080/20229
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.17.x
Gerrit-MessageType: newchange
Gerrit-Change-Id: I929d54d22c1c938ee67fdda9f4c2bb68c028b4ec
Gerrit-Change-Number: 20229
Gerrit-PatchSet: 1
Gerrit-Owner: Yingchun Lai <laiyingc...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <achenn...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <ale...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to