splatch commented on PR #10337: URL: https://github.com/apache/ignite/pull/10337#issuecomment-1287494373
Hello @SammyVimes, thank you for picking it up and following it. I was afraid of submitting this PR as I was not able to run all the tests, hence I was not able to confirm it prior doing official attempt. I thought of bridging `GridSpringResourceContext` to updated API and I think its entirely doable with adapter I attempted to draw in ignite-spring module. We just need to map `@SpringResource` and `@SpringApplicationContext` annotations to the `GridInjectResourceContext`. One note - I am not certain about use `@SpringApplicationContext` and if we will be able to cover its all use cases, as it was carried a bit differently than regular beans. From compilation point of view it should work, because most of ioc stuff was private to ignite-core so far. If you can hold for few days I will use this time to make a prototype of injection registry working under osgi to confirm whole concept works beyond Spring. I will be able to contribute other thing to `ignite-osgi-ext` in later step. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
