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

Bump Elasticsearch spark version to 8.2.3 #126

Merged
merged 2 commits into from Mar 6, 2024

Conversation

sylvainOL
Copy link
Contributor

Current version of elasticsearch-spark-20 jar is using a 7.x version and doesn't want to work with Elasticsearch 8.x.

Which problem is this PR solving?

Short description of the changes

  • Bumping version of jar to make it also work with 8.x versions.

@pavolloffay
Copy link
Member

and doesn't want to work with Elasticsearch 8.x.

If this issue is resolving ES8 compatibility then there should be e2e test for ES 8 e.g. https://github.com/jaegertracing/spark-dependencies/blob/main/.github/workflows/ci.yaml#L25

Current version of elasticsearch-spark-20 jar is using a 7.x version and
doesn't want to work with Elasticsearch 8.x.
Bumping version of jar to make it also work with 8.x versions.

Signed-off-by: Sylvain Desbureaux <37377804+sylvainOL@users.noreply.github.com>
@sylvainOL
Copy link
Contributor Author

I've added a test for es8.

But I'm not sure at all it'll work with es 5.6.9 which is the default version...

@pavolloffay
Copy link
Member

But I'm not sure at all it'll work with es 5.6.9 which is the default version...

We should find out. Are you using Jaeger with ES8 jaegertracing/jaeger#3571?

@s7an-it
Copy link

s7an-it commented Aug 24, 2022

@sylvainOL, what is the state of this change, can we say prod backend ES 8 is ok after this if you run it?

@sergiomcalzada
Copy link

Any update on this?

@sylvainOL
Copy link
Contributor Author

Hello,
it doesn't seems to work sorry :(

@desert3agle
Copy link

does spark jobs work with opensearch?

@piratetm
Copy link

up

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.

Support for Elasticsearch 8.x
7 participants