Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/12879 )
Change subject: WIP: [backup] Add incremental backup/restore support ...................................................................... Patch Set 1: This looks good. To address the potential conflict where a table is renamed and then recreated, we can allow both the renamed table and the newly created table to coexist in the same root dir by naming the directory structure for a table with the table id as a component and storing both current table name and table id in a metadata file. We can document the desired behavior of a given naming conflict as: if two tables had the same name, they never had the same name at the same time. When restoring a table from backup by name and timestamp, we select the backup to restore as the backup of the table with that name at that time. -- To view, visit http://gerrit.cloudera.org:8080/12879 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I50b21d921fefbf4d7e8bd1766285258e8014d890 Gerrit-Change-Number: 12879 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke <granthe...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com> Gerrit-Comment-Date: Wed, 03 Apr 2019 04:50:47 +0000 Gerrit-HasComments: No