You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Tony Foerster (Code Review)" <ge...@cloudera.org> on 2018/07/19 20:33:38 UTC

[kudu-CR] Add error handling to Backup and Restore

Hello Kudu Jenkins, Grant Henke, 

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

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

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

Change subject: Add error handling to Backup and Restore
......................................................................

Add error handling to Backup and Restore

A backup/restore will be attempted on every table.  If any one table
fails job will exit with a non-zero code.

If a table is successfully backed up in the current job or the table
metadata has a timestamp up to the current user supplied timestamp, the table
will be marked successful.
If a table fails to back up any partial data will be removed. If a table fails
to restore and a restore table has already been created, it will not be
removed.

At the end of the job if any table has failed to backup/restore an
exception is thrown and the job exits with a non-zero code.

Spark SaveMode overwrite/append semantics are not handled as part of this
commit.

Change-Id: I104fd604e12b70fcff9beace71ed4ef96b54d691
---
M java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala
M java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackupOptions.scala
M java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduRestore.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/TestContext.scala
5 files changed, 368 insertions(+), 159 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/41/10941/5
-- 
To view, visit http://gerrit.cloudera.org:8080/10941
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I104fd604e12b70fcff9beace71ed4ef96b54d691
Gerrit-Change-Number: 10941
Gerrit-PatchSet: 5
Gerrit-Owner: Tony Foerster <an...@gmail.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tony Foerster <an...@gmail.com>