-
Notifications
You must be signed in to change notification settings - Fork 41
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
Add an example search handler #7
Comments
Sounds interesting, but I'm not sure I follow. What is a Handler in this context? |
Server side solr, in solrconfig.xml <requestHandler name="/search" class="solr.SearchHandler">
(...)
</requestHandler> in there you can config your defaults for facets, spellcheck, highlighter etc.. |
Oh- I see :) Good point! I really want Solrstrap to work with an "out of the box" Solr installation, so that it works as quickly as possible for the most possible people- even if it is kind of messy. So anything that involves a change to solrconfig.xml is (at the moment) out. |
Then you need to make every option available in the getSearchURL function, which in most cases is not what you want. |
The way it is set up now definitely sacrifices code-prettiness in getSearchURL for ease of setup. Although getSearchURL is moderately hairy its not totally spaghetti (yet). Setting a SearchHandler would make getSearchURL simpler and the solconfig.xml more complex. |
You make it sound like having a good configured searchHandler is a bad thing? |
I was thinking, would it be an idea to add an example search handler?
That way you have an example for the solr implementation and the clientside implementation. As a bonus you can minimize the getSearchURL function :)
The text was updated successfully, but these errors were encountered: