lw309637554 commented on pull request #2379: URL: https://github.com/apache/hudi/pull/2379#issuecomment-755243599
> LGTM. @n3nash or @leesf would you mind taking a final pass and merge this? @satishkotha @n3nash @leesf thanks , happy new years! Have resolved all the comments. @satishkotha Also have some questions for you. Why clustering instant time should greater than latest commit? I see BaseScheduleCompactionActionExecutor.execute() have check that should not have earliest write inflight instant time than compaction instant time. But BaseClusteringPlanActionExecutor.execute() do not have similarity check. Thanks. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
