Quite a few enums have accumulated in the package org.hibernate.query.sqm. At least one of them, NullOrdering just clearly doesn't belong there. But there are some others which while relevant to the sqm package have also leaked out onto other API layers including the org.hibernate.query.criteria package. Since this package is actually some sort of abstraction over org.hibernate.query.sqm, this is a layer-breaker. Luckily, org.hibernate.query.sqm is marked @Incubating so we can move these enums out of there. I’m not quite sure where to move them, however. It might not be a great idea to accumulate too much stuff in org.hibernate.query. Not sure. |