[hibernate-dev] [ORM] About making delegating implementations abstract

Hi, While updating OGM to use ORM 5.2, I found out that the delegating implementations of a few classes are missing methods: - AbstractDelegatingSessionBuilder - AbstractDelegatingSessionFactoryBuilder It stayed unnoticed because the classes are abstract so they don't complain about missing methods. Not sure what to do about this. At least one implementation of this sort of things is not abstract - SessionDelegatorBaseImpl - and, apparently, it allowed to detect missing methods. 1/ Should we make all these classes not abstract even if their names make it clear they should be? 2/ Another way to track it would be to have implementations of these abstract classes in the tests. Even unused, it would break the build and warn about this issue. I'm more in favor of 2/ but I thought I might as well ask. (I am preparing a PR to update these classes and also fix the hierarchy by introducing a type parameter where required) -- Guillaume