methodmissing commented on pull request #13026: URL: https://github.com/apache/beam/pull/13026#issuecomment-707050839
I have a branch in progress to run the `io/kafka` tests with more kafka-client versions in https://github.com/apache/beam/compare/master...methodmissing:kafkaio-test-more-versions but haven't had a window to converge on that, but can prioritise otherwise. Related, but forcing a newer kafka-clients version, I think that'll mostly invalidate `ConsumerSpEL` and `ProducerSpEL` as well. @lukecwik thoughts on the implications of bumping default kafka-clients up from 1.0.0 to 2.4.1 in 1 fell swoop? It's runtime pluggable, at Shopify we already use 2.4.1 for a long time and moving towards 2.6.0 for Beam but I worry about backwards compatibility. Also admittedly biased because https://github.com/apache/beam/pull/12794 would have been way less complex to implement and users of `1.0.0` in the wild is setting themselves up for other problems moving forward too 🤷♂️ :smile: cc @sap1ens @vvuibert ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
