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

Updates to latest Zipkin supporting Elasticsearch 8.x #215

Merged
merged 4 commits into from
Dec 30, 2023
Merged

Conversation

codefromthecrypt
Copy link
Member

@codefromthecrypt codefromthecrypt commented Dec 30, 2023

This drops support of Elasticsearch 6.x, including writing to old index
templates, as it is no longer possible with the driver that supports 7.x
and 8.x.

See elastic/elasticsearch-hadoop#2188

This also updates to the latest Spark 3.x version possible, currently
3.3.x. Note: we can't update to Scala 2.13 due to a conflict between
connectors: (DataStax) Cassandra requires Spark 3.4 on Scala 2.13, but
Elastic requires Spark 3.3.

See elastic/elasticsearch-hadoop#2187

This drops support of Elasticsearch 6.x, including writing to old index
templates, as it is no longer possible with the driver that supports 7.x
and 8.x.

See elastic/elasticsearch-hadoop#2188

This also updates to the latest Spark 3.x version possible, currently
3.3.x. Note: we can't update to Scala 2.13 due to a conflict between
connectors: (DataStax) Cassandra requires Spark 3.4 on Scala 2.13, but
Elastic requires Spark 3.3.

See elastic/elasticsearch-hadoop#2187

Signed-off-by: Adrian Cole <[email protected]>
Signed-off-by: Adrian Cole <[email protected]>
@codefromthecrypt
Copy link
Member Author

cc @xeera on notes about ES conflicts, but good news is that 7.x and 8.x pass tests! The main issue is moving up to a more recent scala+spark as that allows JDKs beyond 11 to work.

@codefromthecrypt codefromthecrypt marked this pull request as ready for review December 30, 2023 12:02
Adrian Cole added 2 commits December 30, 2023 20:04
Signed-off-by: Adrian Cole <[email protected]>
Signed-off-by: Adrian Cole <[email protected]>
<!-- TODO: can't update to 2.13 without updating ES driver -->
<!-- Use a common scala binary version, and the highest spark patch version.
Scala 2.13 is held back due to spark version conflicts until Elastic
updates their version. Once 2.13 works, we can also use JDK 17 per
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Theoretically, you can get scala 2.12 to work on later JREs, but SPARK-33772 says it is not advised and to update to 2.13 first

@codefromthecrypt
Copy link
Member Author

going to release this as those trying ES 8 will immediately run into dependencies not working, yet.

@codefromthecrypt codefromthecrypt merged commit 1ee5f70 into master Dec 30, 2023
5 checks passed
@codefromthecrypt codefromthecrypt deleted the ES8 branch December 30, 2023 22:52
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.

1 participant