zhangyue19921010 commented on PR #12514: URL: https://github.com/apache/hudi/pull/12514#issuecomment-2553583458
Thanks for your reviewing danny! > I kind of think we should expose the incremental partitions to the specific XXXStrategy class, because the strategy class can decide the partition filtering itself which should be very related. Totally agree, We need to implement different behaviors such as partition acquisition, partition filtering, and plan construction through different strategies, so that it is more flexible and controllable. But one of the prerequisites for doing so is the need for a unified abstraction of the above API, which is why a `BaseTableServicePlanStrategy` is designed. > Let's clarify the behavior for archived table service commits. Maybe we can do some changes in archive service, such as when Incremental Table Service is enabled, we should always ensure that there is a Clustering/Compaction request metadata in the active timeline Also wen can use `getAllpartitions `as Cover-up plan -- 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]
