prashantwason commented on code in PR #8837: URL: https://github.com/apache/hudi/pull/8837#discussion_r1245809621
########## hudi-client/hudi-client-common/src/main/java/org/apache/hudi/client/BaseHoodieWriteClient.java: ########## @@ -669,32 +669,51 @@ public void restoreToSavepoint() { * @param savepointTime Savepoint time to rollback to */ public void restoreToSavepoint(String savepointTime) { - boolean initialMetadataTableIfNecessary = config.isMetadataTableEnabled(); - if (initialMetadataTableIfNecessary) { + boolean initializeMetadataTableIfNecessary = config.isMetadataTableEnabled(); + if (initializeMetadataTableIfNecessary) { try { - // Delete metadata table directly when users trigger savepoint rollback if mdt existed and beforeTimelineStarts + // Delete metadata table directly when users trigger savepoint rollback if mdt existed and if the savePointTime is beforeTimelineStarts + // or before the oldest compaction on MDT. + // We cannot restore to before the oldest compaction on MDT as we don't have the basefiles before that time. String metadataTableBasePathStr = HoodieTableMetadata.getMetadataTableBasePath(config.getBasePath()); HoodieTableMetaClient mdtClient = HoodieTableMetaClient.builder().setConf(hadoopConf).setBasePath(metadataTableBasePathStr).build(); - // Same as HoodieTableMetadataUtil#processRollbackMetadata + Option<HoodieInstant> latestMdtCompaction = mdtClient.getCommitTimeline().filterCompletedInstants().lastInstant(); + boolean deleteMDT = false; + if (latestMdtCompaction.isPresent()) { + if (HoodieTimeline.LESSER_THAN_OR_EQUALS.test(savepointTime, latestMdtCompaction.get().getTimestamp())) { + LOG.warn(String.format("Deleting MDT during restore to %s as the savepoint is older than oldest compaction %s on MDT", + savepointTime, latestMdtCompaction.get().getTimestamp())); + deleteMDT = true; + } + } + HoodieInstant syncedInstant = new HoodieInstant(false, HoodieTimeline.DELTA_COMMIT_ACTION, savepointTime); // The instant required to sync rollback to MDT has been archived and the mdt syncing will be failed // So that we need to delete the whole MDT here. if (mdtClient.getCommitsTimeline().isBeforeTimelineStarts(syncedInstant.getTimestamp())) { + LOG.warn(String.format("Deleting MDT during restore to %s as the savepoint is older than the MDT timeline %s", Review Comment: Done -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@hudi.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org