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 17:13:25 UTC

[kudu-CR] Add error handling to KuduBackup

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 (#3).

Change subject: Add error handling to KuduBackup
......................................................................

Add error handling to KuduBackup

KuduBackup will attempt to backup every table. If any one table fails the
entire job will fail.

If a table is was already backed up to the specified timestamp or the
table is backed up, the table is marked successful.
If a table fails to back up any partial data will be removed.

At the end of the job if any table has failed to backup/restore an
exception is thrown.

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

Change-Id: I104fd604e12b70fcff9beace71ed4ef96b54d691

Change-Id: I104fd604e12b70fcff9beace71ed4ef96b54d691
---
A java/kudu-backup/out/test/resources/log4j.properties
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
6 files changed, 358 insertions(+), 126 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/41/10941/3
-- 
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: 3
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>