[GitHub] lucene-solr issue #129: LUCENE-7603: Support Graph Token Streams in QueryBui...

2017-01-03 Thread mikemccand
Github user mikemccand commented on the issue: https://github.com/apache/lucene-solr/pull/129 I've merged this into Lucene's master (7.0), and I'm working on 6.x (#130) now. Thanks @mattweber! Can you close this? --- If your project is set up for it, you can reply to this email and

[GitHub] lucene-solr issue #129: LUCENE-7603: Support Graph Token Streams in QueryBui...

2016-12-31 Thread mikemccand
Github user mikemccand commented on the issue: https://github.com/apache/lucene-solr/pull/129 This change looks great to me! What an awesome improvement, to properly use graph token streams at search time so multi-token synonyms are correct. I'll push this in a few days once

[GitHub] lucene-solr issue #129: LUCENE-7603: Support Graph Token Streams in QueryBui...

2016-12-30 Thread mattweber
Github user mattweber commented on the issue: https://github.com/apache/lucene-solr/pull/129 Thanks @dsmiley! I have just pushed up code with your suggestions except for using `BytesRefHash` due to the fact we might have the same `BytesRef` but need a different id because we have

[GitHub] lucene-solr issue #129: LUCENE-7603: Support Graph Token Streams in QueryBui...

2016-12-30 Thread mattweber
Github user mattweber commented on the issue: https://github.com/apache/lucene-solr/pull/129 @mikemccand I addressed you comments. I also added some more tests and fixed a bug that would yield wrong increment when a term that had previously been seen was found again with an

[GitHub] lucene-solr issue #129: LUCENE-7603: Support Graph Token Streams in QueryBui...

2016-12-28 Thread mattweber
Github user mattweber commented on the issue: https://github.com/apache/lucene-solr/pull/129 Rebased against master, added missing ASF header. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have