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 (#13).
Change subject: Add error handling to Backup and Restore
......................................................................
Add error handling to Backup and Restore
A backup/restore will be attempted on every,regardless of any one table
failure. If any one table fails the 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 equaling the 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 Kudu table has already been created the
`-restore` table will not be removed. This should probably be done in a
later commit.
A list of failed/success full backups or restores will be printed at the
end of each job.
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/main/scala/org/apache/kudu/backup/TableMetadata.scala
M java/kudu-backup/src/test/scala/org/apache/kudu/backup/TestKuduBackup.scala
5 files changed, 343 insertions(+), 110 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/41/10941/13
--
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: 13
Gerrit-Owner: Tony Foerster <anthonymfoers...@gmail.com>
Gerrit-Reviewer: Grant Henke <granthe...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tony Foerster <anthonymfoers...@gmail.com>
