SolrJ extension for SolrTextTagger about solrtexttagger HOT 4 OPEN

Very cool! After we switch to Java 8 (which I see this code requires as it uses .stream()), what do you think of providing this code with the text tagger? It would probably result in another module but I think that's better than separate projects altogether.

from solrtexttagger.

I should definitely be an own module (as it only depends on SolrJ and does not have a direct dependency to the SolrTextTagger module. I have no problem of managing it at the above location, but maybe SolrTextTagger user will more likely find it when it is made available under the same groupId as the solr-text-tagger.

Regarding Java 8: I was in the impression that Solr 6 requires Java 8. In SolrJ they renamed some of the main classes with Solr 6. So to support older Java versions one would need to provide two separate releases (one for SolrJ 6 and SolrJ 5)

from solrtexttagger.

I just released 2.3 and bumped the tagger to Solr 6 and Java 8.

Yes, Solr 6 requires Java 8.
Feel free to send a pull request for this contribution. Perhaps the SolrJ module can be a child of the existing one, without having to move the current code to a new subdirectory -- that'd be nice.

from solrtexttagger.

Now that Solr 7.4.0 includes the tagger, perhaps you might want to propose your addition directly to Apache Solr / SolrJ.

from solrtexttagger.