Skip to content
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

Update to Lucene 4.10.4 #25

Closed
wants to merge 3 commits into from
Closed

Update to Lucene 4.10.4 #25

wants to merge 3 commits into from

Conversation

arrdem
Copy link

@arrdem arrdem commented Aug 11, 2015

The Lucene API changed so that IndexWriter.deleteDocuments became variadic. This is a non-issue for Java code since javac will upgrade transparently, but requires some reflection and array work from us. This patch makes the appropriate reflection changes to support the latest 4.X and bumps the patch number.

Porting to 5.X looks like it will be nontrivial.

@technomancy
Copy link
Contributor

Thanks. Please see the discussion in #23. Rather than developing this further, James would rather move work to a fork and promote that, but I don't believe there's any consensus yet about which fork.

@arrdem
Copy link
Author

arrdem commented Aug 12, 2015

Yeah I mailed James earlier this afternoon to ask about this. I think I'm just gonna run with this as suits my needs and there was some noise about becomming the canonical version if it stays active. Closing.

@arrdem arrdem closed this Aug 12, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants