Skip to content

Fix AOT processing for lazy loading jdk proxies. #4352

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
wants to merge 3 commits into from

Conversation

christophstrobl
Copy link
Member

@christophstrobl christophstrobl commented Mar 31, 2023

Closes: #4351
Integration test: spring-projects/spring-aot-smoke-tests#176

Should also be forward ported to main development line.

This commit makes sure to use the ProxyFactory for retrieving the proxied interfaces. This makes sure to capture the exact interface order required when finally loading the proxy at runtime.
mp911de pushed a commit that referenced this pull request Apr 13, 2023
This commit makes sure to use the ProxyFactory for retrieving the proxied interfaces. This makes sure to capture the exact interface order required when finally loading the proxy at runtime.

Original pull request: #4352
Closes #4351
mp911de added a commit that referenced this pull request Apr 13, 2023
Tweak naming.

Original pull request: #4352
See #4351
@mp911de mp911de added the type: bug A general bug label Apr 13, 2023
mp911de pushed a commit that referenced this pull request Apr 13, 2023
This commit makes sure to use the ProxyFactory for retrieving the proxied interfaces. This makes sure to capture the exact interface order required when finally loading the proxy at runtime.

Original pull request: #4352
Closes #4351
mp911de added a commit that referenced this pull request Apr 13, 2023
Tweak naming.

Original pull request: #4352
See #4351
@mp911de mp911de added this to the 4.0.5 (2022.0.5) milestone Apr 13, 2023
@mp911de
Copy link
Member

mp911de commented Apr 13, 2023

That's merged, polished, and backported now.

@mp911de mp911de closed this Apr 13, 2023
@mp911de mp911de deleted the issue/4.0.x/4351 branch April 13, 2023 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A general bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Issue with AOT and Kotlin data class: The following must be non-sealed interfaces
2 participants