Github user mattweber closed the pull request at:
https://github.com/apache/lucene-solr/pull/129
---
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 this feature
enabled and wishes so
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 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
Github user mattweber commented on a diff in the pull request:
https://github.com/apache/lucene-solr/pull/129#discussion_r94171262
--- Diff:
lucene/core/src/java/org/apache/lucene/util/graph/GraphTokenStreamFiniteStrings.java
---
@@ -0,0 +1,294 @@
+/*
+ * Licensed
GitHub user mattweber opened a pull request:
https://github.com/apache/lucene-solr/pull/130
LUCENE-7603: branch_6x Support Graph Token Streams in QueryBuilder
Adds support for handling graph token streams inside the
QueryBuilder util class used by query parsers
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
GitHub user mattweber opened a pull request:
https://github.com/apache/lucene-solr/pull/129
LUCENE-7603: Support Graph Token Streams in QueryBuilder
Adds support for handling graph token streams inside the
QueryBuilder util class used by query parsers.
You can merge this pull