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 (#2).
Change subject: WIP Add error handling to KuduBackup DONT_BUILD KuduBackup will
attempt to backup every table. If any one table fails the entire job will fail.
......................................................................
WIP Add error handling to KuduBackup
DONT_BUILD
KuduBackup will attempt to backup every table. If any one table fails the
entire job will fail.
A backup attempt can have 2 results:
- Success - table was backed up or an existing backup is in the output
dir. Incremental backup to be implemented
- Failure - table failed to backup. Any partial data will be removed
At the end of the job lists of successes and failures are printed out.
An exception is thrown if there are any table failures.
TODO -
- if 'overwrite' flag is specified, overwrite the 'metadata'
folder. Not sure yet how to handle this on retries or if it's even a
good idea.
- more tests
Change-Id: I104fd604e12b70fcff9beace71ed4ef96b54d691
---
A java/kudu-backup/out/test/resources/log4j.properties
A java/kudu-backup/src/main/scala/org/apache/kudu/backup/BackupStatus.scala
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
6 files changed, 232 insertions(+), 56 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/41/10941/2
--
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: 2
Gerrit-Owner: Tony Foerster <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tony Foerster <[email protected]>
