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.

Sign up for GitHub

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

Jump to bottom

Return Elasticsearch warnings only in res.warnings #79

Merged

JoshMock merged 1 commit into elastic:main from pquentin:elasticsearch-warnings

Dec 12, 2023

Member

pquentin commented Dec 8, 2023

Closes #33

I was not able to test this on a real cluster as:

elasticsearch-js does not expose warnings
I did not figure out how to use the transport client directly

But a way to test this is to run a ... 7.17 cluster and close an index:

curl -v -X PUT http://localhost:9200/test-index/
curl -v -X POST http://localhost:9200/test-index/_close

Or to run a 8.12 cluster and use GET _cat/aliases?local=true (elastic/elasticsearch#101815).


          Return Elasticsearch warnings only in res.warnings

db80d0e

pquentin requested a review from JoshMock

December 8, 2023 13:24

JoshMock approved these changes

View reviewed changes

Member

JoshMock left a comment

Looks great! Thanks for the contribution @pquentin.

JoshMock merged commit 027a9d0 into elastic:main

14 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet