Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Bump to Scala 2.13 (including Kafka dependency) #10053

Closed
wants to merge 2 commits into from
Closed

Bump to Scala 2.13 (including Kafka dependency) #10053

wants to merge 2 commits into from

Conversation

gastaldi
Copy link
Contributor

Fixes #10025

@boring-cyborg boring-cyborg bot added area/dependencies Pull requests that update a dependency file area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/kafka area/kafka-streams labels Jun 16, 2020
@gastaldi gastaldi mentioned this pull request Jun 16, 2020
Closed
@gastaldi gastaldi requested a review from gsmet June 16, 2020 14:47
@gastaldi gastaldi marked this pull request as ready for review June 16, 2020 14:48
@gastaldi gastaldi marked this pull request as draft June 16, 2020 18:00
@gsmet
Copy link
Member

gsmet commented Jun 17, 2020

I tried that a while ago and wasn't able to make it work with Debezium.

@gsmet
Copy link
Member

gsmet commented Jun 17, 2020 

2020-06-16T18:00:07.4027409Z Caused by: java.lang.InstantiationError: scala.collection.mutable.ArraySeq
2020-06-16T18:00:07.4028364Z 	at io.debezium.kafka.KafkaServer.startup(KafkaServer.java:218)
2020-06-16T18:00:07.4030095Z 	at java.base/java.util.concurrent.ConcurrentHashMap$ValuesView.forEach(ConcurrentHashMap.java:4772)
2020-06-16T18:00:07.4031034Z 	at io.debezium.kafka.KafkaCluster.startup(KafkaCluster.java:250)
2020-06-16T18:00:07.4032295Z 	at io.quarkus.it.kafka.KafkaSASLTestResource.start(KafkaSASLTestResource.java:44)
2020-06-16T18:00:07.4035990Z 	at io.quarkus.test.common.TestResourceManager.start(TestResourceManager.java:46)
2020-06-16T18:00:07.4036413Z 	... 44 more

@gunnarmorling any chance we could have a Debezium version supporting Scala 2.13?

@gunnarmorling
Copy link
Contributor

I suppose we could do that, but I'm not 100% sure about the consequences. Potentially, it might break other users of KafkaCluster who are bound to 2.12. Note that the Kafka website recommends to use 2.12, if in doubt. Btw. it seems the OP said the problem is resolved on their end?

@gsmet
Copy link
Member

gsmet commented Jul 10, 2020

Closing this one. We can revisit later once Debezium is ready for Scala 2.13.

@gsmet gsmet closed this Jul 10, 2020
@gsmet gsmet added the triage/invalid This doesn't seem right label Jul 10, 2020
@wiradikusuma
Copy link

From https://www.confluent.io/blog/apache-kafka-2-6-updates/

the community has switched to using Scala 2.13 by default and now recommends using Scala 2.13 for production

Can we please upgrade to Scala 2.13 as well?

This was referenced Mar 30, 2022
Merged
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gsmet gsmet Awaiting requested review from gsmet

Assignees
No one assigned
Labels
area/dependencies Pull requests that update a dependency file area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/kafka area/kafka-streams triage/invalid This doesn't seem right
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Can't compile with Scala 2.13
4 participants
@gastaldi @gsmet @gunnarmorling @wiradikusuma