Skip to content

Conversation

@artembilan
Copy link
Member

  • Rework MessagingGatewayRegistrar to parse messaging gateway annotation
    an <gateway> XML using a supplier variant for bean definition.
    Such a feature is required by Spring Native - otherwise we would need to
    register reflection info for to many internal Spring Integration classes
  • Such a change should benefit from regular JDK perspective, too -
    we don't do reflection for this kind of bean registrations

* Rework `MessagingGatewayRegistrar` to parse messaging gateway annotation
an `<gateway>` XML using a supplier variant for bean definition.
Such a feature is required by Spring Native - otherwise we would need to
register reflection info for to many internal Spring Integration classes
* Such a change should benefit from regular JDK perspective, too -
we don't do reflection for this kind of bean registrations
@artembilan artembilan added this to the 5.5.2 milestone Jul 2, 2021
@artembilan
Copy link
Member Author

NOTE: I'll let to know when it is ready for merging.
Didn't have a chance to test it from Spring Native perspective, yet.

@artembilan
Copy link
Member Author

OK. As long as I see this one in native logs:

Caused by: com.oracle.svm.core.jdk.UnsupportedFeatureError: Proxy class defined by interfaces [interface com.example.integration.ControlBusGateway, interface org.springframework.aop.SpringProxy, interface org.springframework.aop.framework.Advised, interface org.springframework.core.DecoratingProxy] not found. 

I can tell that GatewayProxyFactoryBean is created properly.
The Proxy configuration concern is out of scope of the work done in this PR and has to be addressed on Spring Native side.

@artembilan artembilan changed the title [DO NOT MERGE YET] Functional gateway bean definitions Functional gateway bean definitions Jul 2, 2021
@artembilan artembilan requested a review from garyrussell July 2, 2021 20:12
@artembilan
Copy link
Member Author

See Spring Native PR for more info: spring-attic/spring-native#894

@garyrussell garyrussell merged commit db4f120 into spring-projects:main Jul 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants