curcur edited a comment on pull request #16653: URL: https://github.com/apache/flink/pull/16653#issuecomment-891622506
I have a bit concern of this fix: this change swallows all "RejectedExecutionException" no matter it is because of task finish or not. The right way is not to submit mailbox execution after the task finishes, or at least identify this case. I think there should be similar problems for other registered timers, maybe we can also unregister this timer before the task finishes? -- 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: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org