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

upgrade to lucene 4.10.3 release #9100

Closed
wants to merge 1 commit into from
Closed

upgrade to lucene 4.10.3 release #9100

wants to merge 1 commit into from

Conversation

rmuir
Copy link
Contributor

@rmuir rmuir commented Dec 30, 2014

Lucene 4.10.3 is released, switch pom.xml for 1.x to the official release (rather than a snapshot).

@s1monw
Copy link
Contributor

s1monw commented Dec 30, 2014

LGTM

@s1monw
Copy link
Contributor

s1monw commented Dec 30, 2014

@rmuir are you going to push this to 1.4 as well?

@rmuir
Copy link
Contributor Author

rmuir commented Dec 30, 2014

In my opinion, if we upgrade 1.4 branch to 4.10.3, we might need to do a bit more too, e.g. we should consider #8357

@s1monw
Copy link
Contributor

s1monw commented Dec 30, 2014

In my opinion, if we upgrade 1.4 branch to 4.10.3, we might need to do a bit more too, e.g. we should consider #8357

I see - I have to read through the changes / bugfixes for 4.10.3 to tell if it's worth it to be honest

@rmuir
Copy link
Contributor Author

rmuir commented Dec 30, 2014

OK, lets target just 1.5 with this PR? We can open another PR if we want to backport stuff to 1.4, it needs to include #8357 commit along with the pom.xml change, or I think there may be test failures. And honestly it might destabilize 1.4 tests no matter what we do.

@rmuir rmuir added the v1.5.0 label Dec 30, 2014
@s1monw
Copy link
Contributor

s1monw commented Dec 30, 2014

++

rmuir added a commit that referenced this pull request Dec 31, 2014
Upgrades from a 4.10.3 snapshot to the official release.

Closes #9100
@rmuir rmuir closed this Jan 1, 2015
@clintongormley clintongormley added >enhancement :Core/Infra/Core Core issues without another label >upgrade and removed :Core/Infra/Core Core issues without another label labels Mar 19, 2015
@clintongormley clintongormley added :Core/Infra/Core Core issues without another label and removed :Core/Infra/Core Core issues without another label >enhancement labels Aug 25, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Core/Infra/Core Core issues without another label >upgrade v1.5.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants