-
Notifications
You must be signed in to change notification settings - Fork 634
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
SOLR-17057: JSON Query regression of defType #2045
Conversation
If "query" is specified with a String (not JSON structure), "defType" should parse it. Since 9.4 defType was ignored.
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.
Smells like a 9.4.1, yea
+1 for a 9.4.1 release for this issue. I was just upgrading the code for the upcoming AI-Powered Search book, and the most of the book's query examples are now returning no results due to this issue: treygrainger/ai-powered-search#78 Works fine up through Solr 9.3, but most queries are broken with 9.4.0 since we use the edismax query parser through the JSON DSL for most queries. |
# Conflicts: # solr/CHANGES.txt
# Conflicts: # solr/CHANGES.txt
If "query" is specified with a String (not JSON structure), "defType" should parse it. Since 9.4, defType was ignored. Ignoring defType should have been conditional on the query being JSON structured.
If "query" is specified with a String (not JSON structure), "defType" should parse it. Since 9.4, defType was ignored. Ignoring defType should have been conditional on the query being JSON structured.
(a 9.4.1 will happen) |
Thank you @dsmiley ! |
Thanks, @dsmiley - looking forward to it! 🙏 |
If "query" is specified with a String (not JSON structure), "defType" should parse it. Since 9.4 defType was ignored.
https://issues.apache.org/jira/browse/SOLR-17057