Allow for in-place restores. by mattl-netflix · Pull Request #1094 · Netflix/Priam

Change IMetaProxy API to return an ImmutableList of AbstractBackupPaths when fetching incrementals. The iterators are always fully materialized. Also, remove the now-redundant method from BackupRes…

mattl-netflix

committed

aa4dcc1

Include incremental secondary index files in restore.

mattl-netflix

committed

f9de9a7

Remove unused code from our restore tooling.

mattl-netflix

committed

a5a6bb0

Remove waitForCompletion parameter to download method which is always false in practice.

mattl-netflix

committed

6b19184

Remove all non-javadoc comments that don't provide warnings to future maintainers.

mattl-netflix

committed

71756d1

Remove redundant logging.

mattl-netflix

committed

50b41cb

Remove redundant vertical whitespace.

mattl-netflix

committed

515eb2f

Remove backup racs as that configuration is not used in practice and is not relevant to whether a restore should be attempted.

mattl-netflix

committed

948485a

Consolidate restore status API.

mattl-netflix

committed

78aca40

Cease wiping the data directory in advance of restore. It is counter-intuitive and will be empty in practice in most cases anyway.

mattl-netflix

committed

78dae29

Move getBackupDirectories to BackupRestoreUtil and change its signature to cease requiring an IConfiguration but rather just take the data directory path as a String.

mattl-netflix

committed

a7c0975

Cease stopping Cassandra in the restore process and reveal the ability to restore to a staging area and then import or move the data depending on whether Cassandra is running.

mattl-netflix

committed

71c045b

Rename file variable to be idomatic Java.

mattl-netflix

committed

6b1027a

Remove redundant File.exists() calls.

mattl-netflix

committed

728645b

Use Path.resolve rather than switching between Path and String.

mattl-netflix

committed

67a1c0a

Move to canonical method of moving one directory to another in Java.

mattl-netflix

committed

9c3be36

Log failed imports.

mattl-netflix

committed

a1b7e8d

Change policy to not replace existing files by default when doing an offline restore. Remove unused code.

mattl-netflix

committed

e1ebaa5

Navigation Menu

Allow for in-place restores. - #1094

Allow for in-place restores.#1094
mattl-netflix wants to merge 18 commits into
4.xfrom
feature/in_place_restore_4

Commits on Mar 17, 2025

Change IMetaProxy API to return an ImmutableList of AbstractBackupPaths when fetching incrementals. The iterators are always fully materialized. Also, remove the now-redundant method from BackupRes…

Include incremental secondary index files in restore.

Remove unused code from our restore tooling.

Remove waitForCompletion parameter to download method which is always false in practice.

Remove all non-javadoc comments that don't provide warnings to future maintainers.

Remove redundant logging.

Remove redundant vertical whitespace.

Remove backup racs as that configuration is not used in practice and is not relevant to whether a restore should be attempted.

Consolidate restore status API.

Cease wiping the data directory in advance of restore. It is counter-intuitive and will be empty in practice in most cases anyway.

Move getBackupDirectories to BackupRestoreUtil and change its signature to cease requiring an IConfiguration but rather just take the data directory path as a String.

Cease stopping Cassandra in the restore process and reveal the ability to restore to a staging area and then import or move the data depending on whether Cassandra is running.

Rename file variable to be idomatic Java.

Remove redundant File.exists() calls.

Use Path.resolve rather than switching between Path and String.

Move to canonical method of moving one directory to another in Java.

Log failed imports.

Change policy to not replace existing files by default when doing an offline restore. Remove unused code.