Gavin King (
https://hibernate.atlassian.net/secure/ViewProfile.jspa?accountId=557058%...
) *commented* on HHH-16858 (
https://hibernate.atlassian.net/browse/HHH-16858?atlOrigin=eyJpIjoiMjEyYj...
)
Re: Can't compare test expression of type [BasicSqmPathSource(type : Enum)] with
element of type [basicType(a)5(java.lang.Integer,4)] (
https://hibernate.atlassian.net/browse/HHH-16858?atlOrigin=eyJpIjoiMjEyYj...
)
So the reason I’m suspicious of this, and think it would be totally reasonable if we
simply disallowed it is that if you use a STRING mapped enum, which I frankly think makes
more sense most of the time, then < would be doing an alphabetic string comparison,
which simply doesn’t seem like it could ever be meaningful.
I guess it sorta might make sense for some ORDINAL mappings, but even that is pretty
doubtful.
So I guess my vote would be to disallow this with a more meaningful error, but I would not
strongly object to allowing it only for ordinal-mapped enums.
(
https://hibernate.atlassian.net/browse/HHH-16858#add-comment?atlOrigin=ey...
) Add Comment (
https://hibernate.atlassian.net/browse/HHH-16858#add-comment?atlOrigin=ey...
)
Get Jira notifications on your phone! Download the Jira Cloud app for Android (
https://play.google.com/store/apps/details?id=com.atlassian.android.jira....
) or iOS (
https://itunes.apple.com/app/apple-store/id1006972087?pt=696495&ct=Em...
) This message was sent by Atlassian Jira (v1001.0.0-SNAPSHOT#100227- sha1:3bc807d )