On Tue, May 16, 2017 at 11:20 AM Sanne Grinovero <sanne(a)hibernate.org>
wrote:
Since 6 is coming I'd like to suggest two trivial changes for
the
listeners:
# Serializable
I assume we can drop the "extend Serializable" from all these interfaces ?
Personally I've never been a fan of interfaces extending Serializable, so I
am fine with that.
# PostInsertEventListener.requiresPostCommitHanding(EntityPersister)
I suspect there is a typo in this method?
N.B. there are some more copies of `requiresPostCommitHanding` in
other listeners, beyond the PostInsertEventListener.
Fine by me, but I wonder if we ought to deprecate it instead and use
default methods. Or maybe do that retro in 5.2 and do clean in 6