gharris1727 commented on PR #14101: URL: https://github.com/apache/kafka/pull/14101#issuecomment-1656189960
Hey thanks @akitoshka for giving some attention to this issue. Unfortunately the fix as-is can't be merged, for the reasons that @yashmayya and @vamossagar12 raised above. While the particular plugin behavior (resource leaks) is out-of-scope for the framework to fix, I do think it would be appropriate for the framework to indicate to plugins that they have been cancelled, to allow themselves to clean up. There is a relevant ticket here: https://issues.apache.org/jira/browse/KAFKA-14725 which covers some of the suggested fixes, and I'm happy to review a PR for that. Thanks again for your contribution! -- 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: jira-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
