vbhanuchander-lang commented on PR #309: URL: https://github.com/apache/flink-connector-kafka/pull/309#issuecomment-5611485690
@MartijnVisser tagging you since you reported FLINK-40586 — this is the enumerator-side fix for the `testIdleReader` failover you saw in the weekly CI. One thing worth flagging: `Flink Connector Kafka CI` is sitting at `action_required`, since this is my first PR in this repo, so it needs a maintainer to approve the workflow run before anything actually compiles or tests here. Only `Title Validator` has run so far. I verified locally against Flink 2.2.1 on **JDK 21 only** — `DynamicKafkaSourceEnumeratorTest` is 31/31 with the change, and the new test fails on unfixed `main` with the first reader signalled 3 times rather than once — so the JDK 11 and 17 legs are still unverified. If you or another Kafka committer could also assign the ticket to me, happy to keep it tidy. No rush on either; mainly noting the CI gate so the missing checks don't read as checks that passed. -- 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]
