HHH-19523 correctly initialize Pre/PostCollectionXxxxEvents from StatelessSession

[gavinking]

@DavideD note that this change breaks Hibernate Reactive

[Please describe here what your change is about]

---

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license
and can be relicensed under the terms of the LGPL v2.1 license in the future at the maintainers' discretion.
For more information on licensing, please check here.

---

https://hibernate.atlassian.net/browse/HHH-19523

[DavideD]

ACK

[DavideD]

No test case?

[gavinking]

No test case?

Hold yr horses, I'm working on it.

[DavideD]

Cool, thanks

[yrodiere]

@DavideD I will need to backport this one to 7.0... Could you please work on a patch to adjust Reactive accordingly in 3.0?

[gavinking]

I will need to backport this one to 7.0... Could you please work on a patch to adjust Reactive accordingly in 3.0?

Oooh I don't like this idea. This ties micro versions of Reactive to micro versions of ORM.

[yrodiere]

I will need to backport this one to 7.0... Could you please work on a patch to adjust Reactive accordingly in 3.0?

Oooh I don't like this idea. This ties micro versions of Reactive to micro versions of ORM.

I hate it as much as you do. Do you think this can be fixed another way?

[gavinking]

Do you think this can be fixed another way?

You could just disable the code in StatelessSessionImpl which raises those events. Since, quite clearly, they're not doing any useful work.

[yrodiere]

Do you think this can be fixed another way?

You could just disable the code in StatelessSessionImpl which raises those events. Since, quite clearly, they're not doing any useful work.

Fine by me. @mbellade , since you were already looking into this... could you please try it? And add the test from this PR, or another one if necessary?

I'd do it, but I'm fighting with website config.

[mbellade]

Here's the pr: #10465.