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

Add slices & wait_for_active_shards to UpdateByQuery whitelist #843

Closed
wants to merge 2 commits into from

Conversation

SosthenG
Copy link

@SosthenG SosthenG commented Mar 4, 2019

As specified by the doc

@thePanz
Copy link
Contributor

thePanz commented Mar 15, 2019

Tests failures are not related to the changes introduced in this PR

@thePanz
Copy link
Contributor

thePanz commented Mar 25, 2019

@polyfractal any news on this PR?

@SosthenG SosthenG changed the title Add slices to UpdateByQuery whitelist Add slices & wait_for_active_shards to UpdateByQuery whitelist Apr 5, 2019
@thePanz
Copy link
Contributor

thePanz commented Jul 4, 2019

@polyfractal any updates?

@polyfractal
Copy link
Contributor

Hey @thePanz, @ezimuel has taken over maintaining this library (and has been much better about keeping it updated than I did!). He's working on a backlog of issues right now but I'm sure this will get taken care of at some point in the near future.

Sorry for the delay!

@ezimuel
Copy link
Contributor

ezimuel commented Jul 4, 2019

@thePanz, @polyfractal the slices and wait_for_active_shards parameters have been added in release 7.0.0.

@ezimuel ezimuel closed this Jul 4, 2019
@thePanz
Copy link
Contributor

thePanz commented Jul 4, 2019

@ezimuel I see that you introduced those in #897 (the diff is not clean, as it contains a huge mix of php-CS fixes, bc-break of supported PHP version and support for new API parameters).

In any case: those two parameters (slices and wait_for_active_shards) are available from ES 5.6: https://www.elastic.co/guide/en/elasticsearch/reference/5.6/docs-update-by-query.html#docs-update-by-query-slice and https://www.elastic.co/guide/en/elasticsearch/reference/5.6/docs-update-by-query.html#_url_parameters_2

Does it means that the changes in the API parameters must be back-ported to the 5.6 branch?

@ezimuel
Copy link
Contributor

ezimuel commented Jul 4, 2019

ES 5.6 reaches EOL on 2019-03-11 and it's not maintained since 7.0.0. See Elastic Product End of Life Dates for more information.
This means, I can apply your PR to 6.0, 6.5.x and 6.7.x branches, for releasing 6.1.1, 6.5.1 and 6.7.2.
@thePanz are you using Elasticsearch 5, 6 or 7?

@ezimuel ezimuel reopened this Jul 4, 2019
@thePanz
Copy link
Contributor

thePanz commented Jul 9, 2019

@ezimuel I am still on ES 5.6.
So are you planning to drop support for the 5.x branch of this library?

@ezimuel
Copy link
Contributor

ezimuel commented Jul 11, 2019

@thePanz officially, I cannot fully support a library to connect to a server that is not supported anymore. That said, I would like to help all the users of this library. In this case, I'll release it for 5.x and 6.x.

@ezimuel
Copy link
Contributor

ezimuel commented Jul 18, 2019

@thePanz I just released elasticsearch-php 5.5.0 that fixes this issue and also #846.

@ezimuel ezimuel closed this Jul 18, 2019
@thePanz
Copy link
Contributor

thePanz commented Jul 18, 2019

Cool! Thanks!

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.

4 participants