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

Elastic minimum version requirement #9

Closed
rpalcolea opened this issue May 17, 2022 · 7 comments
Closed

Elastic minimum version requirement #9

rpalcolea opened this issue May 17, 2022 · 7 comments

Comments

@rpalcolea
Copy link

rpalcolea commented May 17, 2022

Hi @breskeby

Wonder what are your thoughts on the following

I see you have

elasticsearch-java:8.1.2

And also noticed the following when trying with an old version of elastic:

[es/indices.exists] Missing [X-Elastic-Product] header. Please check that you are connecting to an Elasticsearch instance, and that any networking filters are preserving that header.

Looks like this was introduced back in 8.x elastic/elasticsearch@c06608d and I guess related to elastic/elasticsearch#73424

Wondering if either:

  1. The java client could be downgraded to support older versions of elastic (I wish we all could run with latest and greatest 😭 ) OR
  2. Call out in the README that ElasticSearch 8 might be the minimum supported version?

Thoughts?

@breskeby
Copy link
Collaborator

I could definitely update the docs. But supporting elastic 7.x is quite some effort I think

@rpalcolea
Copy link
Author

I'll close this then :) thanks!

@breskeby breskeby reopened this May 17, 2022
@breskeby
Copy link
Collaborator

@rpalcolea Actually I took a look and I could try using an older elasticsearch java client version as this client is meant to be forward compatible. which elasticsearch version do you use?

@rpalcolea
Copy link
Author

oh that would be nice. We use 5, 6 and possibly 7 if necessary

@breskeby
Copy link
Collaborator

I've changed the java client to be 7.17.3. That should provide 7.x support. 5 and 6 is just to much hassle given those elasticsearch versions are quite outdated

@rpalcolea
Copy link
Author

nice! I'll keep an eye for the next release then! thanks again!

@breskeby
Copy link
Collaborator

@rpalcolea Can you give 0.9.2 a try?

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

No branches or pull requests

2 participants