Skip to content

Deprecate Spark 2.x #2305

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

Merged
merged 6 commits into from
Jan 3, 2025
Merged

Deprecate Spark 2.x #2305

merged 6 commits into from
Jan 3, 2025

Conversation

samxbr
Copy link
Contributor

@samxbr samxbr commented Dec 24, 2024

Apache Spark 2.x has reached end of support since 2021, we are deprecating Spark 2.x in 8.18. Support for it will be removed in 9.0. This change will be backported to 8.x branch.

@samxbr samxbr changed the title Trigger CI Deprecate Spark 2.x Dec 24, 2024
@samxbr samxbr requested review from jbaiera and masseyke January 2, 2025 21:05
@samxbr samxbr marked this pull request as ready for review January 2, 2025 21:05

==== Spark 2.x support is deprecated

Spark 2.x is no longer maintained. Spark 3 are still supported.
Copy link
Member

Choose a reason for hiding this comment

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

Spark 3 is still supported

sc.parallelize(rawCore, 1).saveToEs(target)
val qjson =
"""{"query":{"range":{"colint":{"from":null,"to":"9","include_lower":true,"include_upper":true}}}}"""
"""{"query":{"range":{"colint":{"lte":"9"}}}}"""
Copy link
Member

Choose a reason for hiding this comment

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

I assume this was some change in range query syntax in Elasticsearch, not related to deprecation?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Exactly, this is to fix the broken test.

Copy link
Member

@masseyke masseyke left a comment

Choose a reason for hiding this comment

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

LGTM

@samxbr samxbr merged commit e4af476 into elastic:main Jan 3, 2025
3 checks passed
samxbr added a commit to samxbr/elasticsearch-hadoop that referenced this pull request Jan 3, 2025
(cherry picked from commit e4af476)
@samxbr
Copy link
Contributor Author

samxbr commented Jan 3, 2025

💚 All backports created successfully

Status Branch Result
8.x

Questions ?

Please refer to the Backport tool documentation

samxbr added a commit that referenced this pull request Jan 3, 2025
@samxbr samxbr mentioned this pull request Jan 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants