SOLR-18100: Improve Combined Query Feature documentation#4144
SOLR-18100: Improve Combined Query Feature documentation#4144ilariapet wants to merge 5 commits intoapache:mainfrom
Conversation
…combined-query-documentation
solr/solr-ref-guide/modules/query-guide/pages/json-combined-query-dsl.adoc
Outdated
Show resolved
Hide resolved
solr/solr-ref-guide/modules/query-guide/pages/json-combined-query-dsl.adoc
Show resolved
Hide resolved
alessandrobenedetti
left a comment
There was a problem hiding this comment.
The documentation change is interesting and needs some minor changes
dsmiley
left a comment
There was a problem hiding this comment.
thanks for contributing!
solr/solr-ref-guide/modules/query-guide/pages/json-combined-query-dsl.adoc
Outdated
Show resolved
Hide resolved
solr/solr-ref-guide/modules/query-guide/pages/json-combined-query-dsl.adoc
Outdated
Show resolved
Hide resolved
solr/solr-ref-guide/modules/query-guide/pages/json-combined-query-dsl.adoc
Outdated
Show resolved
Hide resolved
solr/solr-ref-guide/modules/query-guide/pages/json-combined-query-dsl.adoc
Outdated
Show resolved
Hide resolved
changelog/unreleased/SOLR-18100-improve-combined-query-documentation.yml
Outdated
Show resolved
Hide resolved
| [IMPORTANT] | ||
| ==== | ||
| This feature works in both Standalone and SolrCloud modes and always performs distributed search execution. | ||
| In Standalone (user-managed) mode, shard URLs must be explicitly allow-listed using the *allowUrls* parameter, otherwise Solr returns HTTP 403. For example: |
There was a problem hiding this comment.
I haven’t encountered this issue before, as it might vary depending on the infrastructure (?)
There was a problem hiding this comment.
I tested this feature while preparing a tutorial blog post (an end-to-end walkthrough explaining how to use it). I ran everything locally. When I used Solr in standalone mode and executed the combined query, I received the error. Could you please try to reproduce it on your side?
There was a problem hiding this comment.
It’s better to mention it IMO, provided you can replicate it.
solr/solr-ref-guide/modules/query-guide/pages/json-combined-query-dsl.adoc
Show resolved
Hide resolved
|
@dsmiley, @ercsonusharma and @alessandrobenedetti Thank you very much for the review. |
…combined-query-documentation
https://issues.apache.org/jira/browse/SOLR-18100
Description
Some sections of the Combined Query Feature documentation have been made clearer to improve understanding and usability of the feature.
Checklist
Please review the following and check all that apply:
mainbranch../gradlew check.