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
Expose dist
/pre
/post
options for SpanNotQuery
#4452
Conversation
Merged latest master (broke on ElasticSearch -> Elasticsearch renaming) |
Will attempt to rebase soon. I will also try to incorporate the test cases from the Lucene repo |
Hi @brusic - just pinging to find out if you've had a chance to look at this yet. |
I thought the ES crew forgot about this PR. :) I will be traveling this On Mon, Oct 20, 2014 at 5:49 AM, Clinton Gormley notifications@github.com
|
+1 to Expose dist/pre/post options for SpanNotQuery |
…ror condition. Set appropriate defaults for pre/post.
Rebased with latest master |
Thanks @brusic, although it took a while, it looked good and I pulled this in. |
@brusic Thanks for this fix and the answers on ES group. |
dist
/pre
/post
options for SpanNotQuery
Lucene 4.5 provides more options for the SpanNotQuery. Not sure if to update the docs with the pull request or not. If accepted and I knew the potential elasticsearch version which might contain it, I can add the appropriate "Added in ..." message in the documentation.