Closed pjgg closed 2 years ago
/cc @cescoffier, @ozangunalp
Kafka extension's kafka-avro-serializer
support has only been tested on version 6.1.1
With the latest version of confluent schema-registry we need to register for reflection the class io.confluent.kafka.serializers.context.NullContextNameStrategy
and maybe others.
Kafka-avro-serializers 6.2.1 is also covered by this testsuite
But yes definitely I think that we should also support in native mode the newest confluent serializer library. Please let me know if you need some support in order to move on this issue.
I should be able to fix it. Is there a way to extract a reproducer from your test suite? It would be great to have something in Quarkus itself.
@cescoffier Changing https://github.com/quarkusio/quarkus/blob/main/integration-tests/kafka-avro-apicurio2/pom.xml#L68 to 7.0.0 should reproduce the issue in the test with native compilation.
Describe the bug
Quarkus version:
999-SNAPSHOT
Reproducer: https://github.com/quarkus-qe/quarkus-test-suite And this serializer:cmd:
mvn clean verify -Pmessaging-modules -Dnative -Dit.test=ConfluentKafkaAvroIT -pl messaging/kafka-avro-reactive-messaging
When I am running Kafka Confluent + AVRO test, I am getting the following exception:
Note: if you are going to reproduce it following the above steps double-check that the scenario is not disabled for native mode. Remove
@DisabledOnNative
on the following classWorks as expected in JVM mode. Also, the same test works in Strimzi in JVM and native mode. This scenario works with
io.confluent:kafka-avro-serializer:6.2.1
Expected behavior
Same behavior in JVM/Native in Strimzi/Confluent
Actual behavior
Confluent native mode doesn't work with
io.confluent:kafka-avro-serializer:7.0.0
How to Reproduce?
No response
Output of
uname -a
orver
No response
Output of
java -version
No response
GraalVM version (if different from Java)
No response
Quarkus version or git rev
No response
Build tool (ie. output of
mvnw --version
orgradlew --version
)No response
Additional information
No response