Open
Description
openedon Feb 5, 2021
Description
There are currently multiple problems with the ability to inject a Stage.Session
or a Mutiny.Session
related with, and not only:
- the fact that such beans get disposed before the execution of the operation
- the fact that some operations are delegated to a different thread, leading for the ArC proxy to return a different instance from its threadlocal
While we can (and should) be able to fix all these issues, it is causing many problems at the moment so I feel it was a mistake to already expose such producers.
This is a breaking change, but the module is flagged as experimental.
Also, it's trivial for people to inject the Stage.SessionFactory
| Mutiny.SessionFactory
instead, and use it to open sessions as needed. IMO, it's also looking more idiomatic for reactive code.
Implementation ideas
Remove the producers from the project.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment