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

Managed types for Neo4j are not used in Neo4j Data auto configuration. #37573

Closed
michael-simons opened this issue Sep 25, 2023 · 2 comments
Closed
Labels
status: superseded An issue that has been superseded by another

Comments

@michael-simons
Copy link
Contributor

The auto configuration for imperative and reactive Spring Data variant still uses

Set<Class<?>> initialEntityClasses = new EntityScanner(applicationContext).scan(Node.class,
				RelationshipProperties.class);

as initial entity classes for the mapping context. This works fine in JVM mode and to some extent in native mode. It fails however with certain inheritance scenarios (See spring-projects/spring-data-neo4j#2798).

The solution is to use populate the mapping context with the managed entities and I'll have a PR up in a moment.

@wilkinsona
Copy link
Member

Closing in favor of #37574. Thanks for the PR, @michael-simons.

@wilkinsona wilkinsona closed this as not planned Won't fix, can't repro, duplicate, stale Sep 25, 2023
@wilkinsona wilkinsona added status: superseded An issue that has been superseded by another and removed status: waiting-for-triage An issue we've not yet triaged labels Sep 25, 2023
@michael-simons
Copy link
Contributor Author

Sure, my pleasure. And sorry for not understanding how this is supposed to work to begin with.

michael-simons added a commit to michael-simons/spring-boot that referenced this issue Sep 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: superseded An issue that has been superseded by another
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants