Skip to content

Add Support for Spring AOT #2359

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

Closed
garyrussell opened this issue Jul 20, 2022 · 0 comments · Fixed by #2364
Closed

Add Support for Spring AOT #2359

garyrussell opened this issue Jul 20, 2022 · 0 comments · Fixed by #2364

Comments

@garyrussell
Copy link
Contributor

Migrate hints from spring-native.

@garyrussell garyrussell added this to the 3.0.0-M6 milestone Jul 20, 2022
garyrussell added a commit to garyrussell/spring-kafka that referenced this issue Jul 21, 2022
Resolves spring-projects#2359

This is sufficient for the basic Kafka smoke test.

I will issue another PR if changes are needed for the Streams and Avro versions.
garyrussell added a commit to garyrussell/spring-kafka that referenced this issue Jul 21, 2022
Resolves spring-projects#2359

This is sufficient for the basic Kafka smoke test.

I will issue another PR if changes are needed for the Streams and Avro versions.
I have a `KafkaAvroBeanRegistrationAotProcessor` but I have omitted it from
this commit because it hasn't been tested.
artembilan pushed a commit that referenced this issue Jul 27, 2022
Resolves #2359

This is sufficient for the basic Kafka smoke test.

* Remove duplicates; register annotations properly; remove avro from factories.
* Remove `@MessageMapping`, `@EnableKafka`.
* Fix Spring proxies.
* Add `KafkaAvroBeanRegistrationAotProcessor` to detect Avro types.
* Use `registerTypeIfPresent()` for Kafka Streams.
* Polish Avro detection.
* Polish use of MergedAnnotations.
* Kafka Avro AOT Processor polishing; no need to register for `@KafkaListener` only `GenericMessageListener`.
* Fix `BatchMessageListener` (`List<ConsumerRecord<Foo, Bar>>`).
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.

1 participant