Add/modify classes to adapt compound-splitter for use as a Lucene/Solr p...

[48lizards]

I've made changes to allow the GermanCompoundSplitter to be used as a Solr TokenFilter at index and query time.

• Added a GermanCompoundSplitterTokenFilterFactory class that can be specified in the schema.xml, and a GermanCompoundSplitterTokenFilter class with a decompose() method that calls GermanCompoundSplitter's split() method.
• Updated the call to FST.findTargetArc() with an extra parameter for compatibility with Lucene 4.x.
• Added slf4j logging, and replaced RuntimeExceptions with logging.

[dweiss]

Hi Jonathan. I will have to revert your commit because you introduced changes that make the code not compile cleanly (missing pom entries). If you're still interested in this, could you submit a patch that runs cleanly with:

mvn clean verify

and

mvn -Pfst

(as described in the README file). Thanks.

[48lizards]

Hi David,

Yes, I can take a look at that this weekend.

Thanks,
Jon

On Friday, January 24, 2014, Dawid Weiss notifications@github.com wrote:

Hi Jonathan. I will have to revert your commit because you introduced
changes that make the code not compile cleanly (missing pom entries). If
you're still interested in this, could you submit a patch that runs cleanly
with:

mvn clean verify

and

mvn -Pfst

(as described in the README file). Thanks.

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/1#issuecomment-33211865
.

[48lizards]

Hi Dawid,

Haven't gotten a chance yet but haven't forgotten.

Thanks,
Jon

On Fri, Jan 24, 2014 at 7:06 AM, Jonathan Beller
beller.jonathan@gmail.comwrote:

Hi David,

Yes, I can take a look at that this weekend.

Thanks,
Jon

On Friday, January 24, 2014, Dawid Weiss notifications@github.com wrote:

Hi Jonathan. I will have to revert your commit because you introduced
changes that make the code not compile cleanly (missing pom entries). If
you're still interested in this, could you submit a patch that runs cleanly
with:

mvn clean verify

and

mvn -Pfst

(as described in the README file). Thanks.

Reply to this email directly or view it on GitHubhttps://github.com//pull/1#issuecomment-33211865
.

[dweiss]

No problem at all, Jon!