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. |
|
