danny0405 commented on code in PR #13216:
URL: https://github.com/apache/hudi/pull/13216#discussion_r2178858651
##########
hudi-client/hudi-client-common/src/main/java/org/apache/hudi/table/action/rollback/CopyOnWriteRollbackActionExecutor.java:
##########
@@ -67,11 +68,23 @@ protected List<HoodieRollbackStat>
executeRollback(HoodieRollbackPlan hoodieRoll
if (instantToRollback.isCompleted()) {
LOG.info("Unpublishing instant " + instantToRollback);
+ table.getMetaClient().getTableFormat().rollback(instantToRollback,
table.getContext(), table.getMetaClient(), table.getViewManager());
+ // Revert the completed instant to inflight in native format.
resolvedInstant = activeTimeline.revertToInflight(instantToRollback);
// reload meta-client to reflect latest timeline status
table.getMetaClient().reloadActiveTimeline();
}
+ // If instant is inflight but marked as completed in native format, delete
the completed instant from storage.
Review Comment:
yeah, let's avoid to listing the timeline multiple times for reqular
workflow when the external table format is not there.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
