Re: [hibernate-dev] v6 and load-event

Hi Steve, looks like you're getting rid of the "event object"? No more events to be allocated? I think that's a great idea, but I wonder about the readability of the code. With "inline types" coming soon to the JDK, the event object types could be great candidates to be converted into inline? If we used the inline types for such things I'm confident that allocation would improve significantly; it's undeniable that not having anything-at-all would be even better - but perhaps it could be a better tradeoff in consideration of maintainability. I also noticed that it's quite possible that some "list of eventlisteners" for a specific event type could be empty: no listeners at all. So an aspect that could be worth exploring while thinking about the event system design, is to avoid creating en event in such cases; there's an example of this approach already in org.hibernate.internal.SessionImpl#internalClear : the signature I had to use is a bit puzzling, but it manages to avoid allocating the event (unless needed) and also avoids allocating the iterator on the listeners. For example PostLoad events by default have a single listener, but looking more closely I believe we could avoid registering that single listener depending on configuration. Thanks, Sanne On Tue, 26 May 2020 at 16:00, Steve Ebersole <steve(a)hibernate.org&gt; wrote: