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

@DomainEvents and @AfterDomainEventPublication not visible on native image #2939

Closed
odrotbohm opened this issue Sep 21, 2023 · 0 comments
Closed

@DomainEvents and @AfterDomainEventPublication not visible on native image #2939

odrotbohm opened this issue Sep 21, 2023 · 0 comments
Assignees
@odrotbohm
Labels
in: core Issues in core support in: repository Repositories abstraction type: bug A general bug

Comments

@odrotbohm
Copy link
Member

In native images, methods annotated with @DomainEvents and @AfterDomainEventPublication are not visible as the processing of aggregate roots are currently does not detect those annotations and registers method with them for reflection.

We should annotate them with @Reflective (similar to @(Reading|Writing)Converter) and make sure we process aggregate root types using ReflectiveRuntimeHintsRegistrar.
@odrotbohm odrotbohm added type: bug A general bug in: core Issues in core support in: repository Repositories abstraction labels Sep 21, 2023
@odrotbohm odrotbohm self-assigned this Sep 21, 2023
odrotbohm added a commit that referenced this issue Sep 21, 2023
@odrotbohm
Fix detection of @DomainEvents and @AfterDomainEventPublication on na…
9bd0abb 
…tive.

We now unconditionally process the aggregate root types declared on repositories for @Reflective annotations, which @de and @adep got meta-annotated with.

Fixes #2939.
odrotbohm added a commit that referenced this issue Sep 21, 2023
@odrotbohm
Fix detection of @DomainEvents and @AfterDomainEventPublication on na…
11f0d24 
…tive.

We now unconditionally process the aggregate root types declared on repositories for @Reflective annotations, which @de and @adep got meta-annotated with.

Fixes #2939.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@odrotbohm odrotbohm

Labels
in: core Issues in core support in: repository Repositories abstraction type: bug A general bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@odrotbohm