swuferhong commented on PR #3174: URL: https://github.com/apache/fluss/pull/3174#issuecomment-4540836800
> It seems that the designed mechanism is somewhat complex and has quite a few configuration variables. Could we simplify the implementation so that doAutoPartition() maintains a PendingDeletePartition set, and an asynchronous deletion thread polls this set, following these rules: > > Only delete one partition at a time; Skip whenever CoordinatorEventQueue is not empty; The thread can poll the event queue every few seconds to look for a deletion opportunity. Hi @platinumhamburg. The previous design was a bit complex and not general enough (e.g. it didn't really cover user-submitted batch drops of tables or partitions), so the new PR abstracts a common class as a throttling queue for replica drop events. Mind taking another look when you have a moment? Thanks a lot! -- 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]
