Skip to content

Conversation

@christophstrobl
Copy link
Member

@christophstrobl christophstrobl commented Sep 15, 2021

We should set the repository type as type hint for the bean container if we can. We should introduce a control mechanism whether we can provide such a hint as the type hint triggers class loading. In some arrangements (e.g. EclipseLink with load-time weaving) we must defer class loading as much as possible.

See also https://github.com/spring-projects-experimental/spring-native/blob/main/spring-aot/src/main/java/org/springframework/data/RepositoryFactoryBeanPostProcessor.java

for further details.

@mp911de
Copy link
Member

mp911de commented Feb 15, 2022

No longer required (context: AOT/Native Object Factory hints).

@mp911de mp911de closed this Feb 15, 2022
@mp911de mp911de deleted the issue/2.5.x/2455 branch February 15, 2022 08:05
@mp911de mp911de added the status: declined A suggestion or change that we don't feel we should currently apply label Feb 15, 2022
@mp911de mp911de restored the issue/2.5.x/2455 branch March 24, 2022 14:38
@mp911de mp911de reopened this Mar 24, 2022
@mp911de mp911de added type: enhancement A general enhancement theme: 3.0 and removed status: declined A suggestion or change that we don't feel we should currently apply labels Mar 24, 2022
@mp911de mp911de added the status: declined A suggestion or change that we don't feel we should currently apply label Sep 16, 2022
@christophstrobl
Copy link
Member Author

obsolete

@mp911de mp911de deleted the issue/2.5.x/2455 branch May 17, 2023 09:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: declined A suggestion or change that we don't feel we should currently apply theme: 3.0 type: enhancement A general enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

RepositoryConfigurationDelegate should provide factoryBeanObjectType as Class instead of String.

3 participants