SteNicholas commented on PR #7669: URL: https://github.com/apache/hudi/pull/7669#issuecomment-1409895036
@voonhous, I had a voice communication with @YannByron. This PR is just a temporary check for deleting partitions. The final reasonable implementation should be to put the check in the table service of compaction or clustering. In other words , table service should not affect the execution of DDL such as deleting partitions. However, it is beneficial to temporarily add the check in this PR. For example, user A creates a Flink task that writes Hudi table 1 and enables asynchronous clustering. At this time, when user B wants to delete partitions, he can Perceived that this partition has a corresponding pending table service, it can at least temporarily refuse user B to delete the partition to avoid affecting user A's Flink task. To sum up, you can merge first, and create a ticket to optimize this check in table service. -- 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]
