Implement boolean searches #11

Closed
onyxfish opened this Issue Oct 3, 2011 · 7 comments

Projects

None yet

2 participants

@onyxfish
Member
onyxfish commented Oct 3, 2011

DataResource.search()

@onyxfish
Member

This works at the solr level via edismax, but doesn't work through the UI.

@onyxfish
Member

Poked at this for a while tonight. The issue doesnt seem to be url encoding as I had speculated. Its also def not Solr config as all edismax query syntax works from the Solr admin.

Current best guess: Sunburnt is doing something to munge the query. It probably has never been tested against edismax.

@onyxfish
Member

Promoting this since the rest of Iteration 7 is done and this is something I can work on without internet. After looking at it last night my thought is that its probably time to punt on Sunburnt (which is not very actively maintained anyway) and simply use the Solr JSON API directly.

@onyxfish
Member

Problem identified: sunburnt treats any query predicate as a single unit so spaces in boolean queries are being escaped. It doesn't look like its possible to force a raw query without losing all the fancy stuff Sunburnt does. Probably more future-proof to just tear it out.

@onyxfish
Member

Done on local master. Will close on push.

This was referenced Nov 22, 2011
@onyxfish
Member

Bonus round: queries like "+tribune -boyer" now work (ie. return Joe for the test case)

@ryanpitts
Member

woot!

@onyxfish onyxfish added a commit that closed this issue Nov 26, 2011
@onyxfish onyxfish Factored out dependency on sunburnt. Replace with custom Solr glue vi…
…a JSON. Closes #10. Closes #11. Paves the way for #9.
53cf4d5
@onyxfish onyxfish closed this in 53cf4d5 Nov 26, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment