New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove Backup Version V1 #1082
Remove Backup Version V1 #1082
Conversation
.findFirst(); | ||
Assert.assertTrue(backupMetadata.isPresent()); | ||
Assert.assertNull(backupMetadata.get().getLastValidated()); | ||
backupStatusMgr.finish(getBackupMetaData(start, Status.FINISHED)); | ||
} | ||
|
||
@Test | ||
public void verifyBackupVersion1DateRange() throws Exception { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It will be good if we can update the function name(remove version1) or remove this test completely as v2 test is already present in verifyBackupVersion2DateRange
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed. The test is redundant and should be removed. I will do that in a follow-up.
@@ -212,11 +212,6 @@ public String getMaxDirectMemory() { | |||
(PRIAM_PRE + ".direct.memory.size.") + instanceInfo.getInstanceType(), "50G"); | |||
} | |||
|
|||
@Override | |||
public String getBackupCronExpression() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this removed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It was only referenced in SnapshotBackup which was removed in this cleanup. The modern equivalent is getSnapshotMetaServiceCronExpression() used in SnapshotMetaTask.
The commits are meant to be reviewed in order to keep the process tractable.
Remove unused CLI tools.
Remove tooling to back up commit logs. It is no longer supported.
Remove IFileSystemContext. It is redundant.
Remove BackupService
Remove BackupVersion.
Remove GCS-specific code. It is no longer in use.
Remove remaining vestiges of backup v1.