Hibernate reactive - remove the Mutiny.Session bean

[mkouba]

• resolves Hibernate reactive - deprecate the usage of injected Mutiny.Session #30717

[yrodiere]

Seems to make sense, but I can't comment on the specifics of Reactive, so I'll let Davide/Sanne do that.

[DavideD]

LGTM

There are several places where we could clean up the code, but it's not that important

[github-actions]

🙈 The PR is closed and the preview is expired.

[mkouba]

There are several places where we could clean up the code, but it's not that important

For sure we could. I've just tried to adapt the original code...

[mkouba]

For the record - I've removed the producer method because there's no easy way to deprecate a bean. I mean we could detect all Mutiny.Session injection points and log a warning during build but a user gets no warning in the IDE etc. So I think it's better to remove the bean completely.

[mkouba]

Note to myself - if this PR is merged then add an appropriate section to the migration guide.

[quarkus-bot]

✔️ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

[Sanne]

For the record - I've removed the producer method because there's no easy way to deprecate a bean. I mean we could detect all Mutiny.Session injection points and log a warning during build but a user gets no warning in the IDE etc. So I think it's better to remove the bean completely.

Makes sense! Many thanks

[Sanne]

Great to remove this!