New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ISPN-6436 Overriding the name of an indexed property is not supported with DSL query #4226
Conversation
1d3ab41
to
0073db2
Compare
There are some familiar commits in this PR, should it be rebased on master instead? |
I'll rebase. I thought I already did that... |
Not sure why github.com UI shows more commits. Should be only 2. Pls fetch again. |
Now it looks ok. Only 2 commits. |
ca42286
to
a86b236
Compare
//todo validate params | ||
for (String paramName : getParameterNames()) { | ||
if (namedParameters.get(paramName) == null) { | ||
throw new IllegalArgumentException("Query parameter '" + paramName + "' was not set"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably we need some internationalization here.
Only some small picky issues. Otherwise looks ok to me. |
Fixed all. |
cf129ab
to
352e351
Compare
Rebased. |
pulling |
merged, thanks @anistor ! |
https://issues.jboss.org/browse/ISPN-6436
https://issues.jboss.org/browse/ISPN-6508