Skip to content

[7.x][ML] Upgrade Gradle Wrapper to use version 6.0.1 #895

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

Merged
merged 1 commit into from
Dec 13, 2019

Conversation

droberts195
Copy link
Contributor

Elasticsearch has recently upgraded to version 6.0.1
(see elastic/elasticsearch#49211) and it's best if
the ml-cpp repo is consistent at least in the major
version of Gradle that is used.

When ml-cpp is built alongside elasticsearch via the
elasticsearch-extra directory it will be using the
Gradle chosen by the elasticsearch repo, so it's
best if that version is also used when building ml-cpp
standalone to avoid integration incompatibilities.

Backport of #891

Elasticsearch has recently upgraded to version 6.0.1
(see elastic/elasticsearch#49211) and it's best if
the ml-cpp repo is consistent at least in the major
version of Gradle that is used.

When ml-cpp is built alongside elasticsearch via the
elasticsearch-extra directory it will be using the
Gradle chosen by the elasticsearch repo, so it's
best if that version is also used when building ml-cpp
standalone to avoid integration incompatibilities.

Backport of elastic#891
@droberts195 droberts195 merged commit 54a989f into elastic:7.x Dec 13, 2019
@droberts195 droberts195 deleted the upgrade_gradle_7x branch December 13, 2019 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant