[GitHub] [flink] tzulitai commented on pull request #22150: [FLINK-31305] Propagate producer exceptions outside of mailbox execut…

["tzulitai (via GitHub)" <gi...@apache.org>]

tzulitai commented on PR #22150:
URL: https://github.com/apache/flink/pull/22150#issuecomment-1480618973

   @tweise on the matter of merge process for this change:
   
   I believe there wasn't any prior announcement for this, but technically now that `release-1.17` branch is cut, and the fact that we've agreed that 1.17 will be the last release where the Flink Kafka Connector will exist in an `apache/flink` core release, we should actually be good now to remove the Flink Kafka connector code from `apache/flink:master`, and going forward the official process would be:
   
   1. Open changes against `apache/flink-connector-kafka:main`
   2. If necessary, backport to previous release branches of `apache/flink-connector-kafka`
   3. While Flink 1.16 and 1.17 is still supported, if appropriate, backport changes to `apache/flink:release-1.16` and `apache/flink:release-1.17` as well.
   
   Merging this PR to `apache/flink:master` now could complicate things a bit with the externalization effort - because technically we can remove the Kafka connector code in `apache/flink:master` any time now.
   
   Perhaps we actually should do that first?


-- 
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-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org