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.

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

Add Avro support to the Kafka connector #9478

Closed
cescoffier opened this issue May 20, 2020 · 2 comments · Fixed by #10213
Closed

Add Avro support to the Kafka connector #9478

cescoffier opened this issue May 20, 2020 · 2 comments · Fixed by #10213

Comments

@cescoffier
Copy link
Member

It should already work, but we need to add a test verifying that Avro can be used.

We should verify using the Confluent registry and the Apicurio registry (and custom serde).

This feature deserves a specific section in the documentation.

Estimation: Quarkus 1.6

@anamarija
Copy link
Contributor

Hi,

If nobody is working on this, can I take a look at this and test everything out.

@cescoffier
Copy link
Member Author

Thanks @anamarija !

I already added a few Avro tests using the Confluent Registry to the Kafka client IT. In this issue, we need to extend these tests with the Apicurio registry (they also provide they own serde). Hopefully this one would work in native (The other Avro does not).

Finally, some tests need to be added to the IT checking the reactive-messaging Kafka connector.

Some references:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants