Skip to content

[LIVY-1064] Initial changes to provide Scala 2.13 compatibility - #530

Merged
gyogal merged 1 commit into
apache:masterfrom
idzikovsky:LIVY-1041-initial-scala-2.13-compatibility
Aug 6, 2026
Merged

[LIVY-1064] Initial changes to provide Scala 2.13 compatibility#530
gyogal merged 1 commit into
apache:masterfrom
idzikovsky:LIVY-1041-initial-scala-2.13-compatibility

Conversation

@idzikovsky

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

I've stared working on Spark 4 compatibility in my local fork. I don't think that all changes that I've made is applicable to be pushed to the upstream project, but this change I'm confident about.

In this fix I mainly updated usage of collections to be compatible with new redesigned Scala 2.13 collections (so as with older Scala 2.11/2.12 collections).

Also, I replaced invocation of ServletContext.initParameters() with ServletContext.setInitParameter(), because the old wrapper is not compatible with a new jakarta.servlet.ServletContext interface (Spark 4 migrated from javax to jakarta), while ServletContext.setInitParameter() is version-independent.

How was this patch tested?

I run unit tests and manually tested that I'm able to create a Session.

Was this patch authored or co-authored using generative AI tooling?

Generated-by: Claude Opus 4.8.

@gyogal

gyogal commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Thank you for your contribution @idzikovsky ! Scala 2.13 compatibility is an important requirement for LIVY-1041 (Add support for Spark 4 via a new Maven profile). Since that JIRA is already assigned to @roczei whose work in progress can be tracked at roczei#3 , could you please create a subtask for your changes (Scala 2.13 support) under LIVY-1041 in the JIRA? That way we could use the new ticket to merge your PR before merging the rest of the code changes required for Spark 4 support (currently being worked on by @roczei).

@idzikovsky idzikovsky changed the title [LIVY-1041] Initial changes to provide Scala 2.13 compatibility [LIVY-1064] Initial changes to provide Scala 2.13 compatibility Aug 5, 2026
@idzikovsky
idzikovsky force-pushed the LIVY-1041-initial-scala-2.13-compatibility branch from 75e137c to 730b10d Compare August 5, 2026 16:58
@idzikovsky

Copy link
Copy Markdown
Contributor Author

Thanks @gyogal for your feedback!

Here is the JIRA subtask:
https://issues.apache.org/jira/browse/LIVY-1064

Also, I've force-pushed the commit with the right commit message.

Anything else that needs to be done?

@gyogal

gyogal commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

The JIRA looks good, thank you for making the updates @idzikovsky!

@gyogal
gyogal merged commit d082174 into apache:master Aug 6, 2026
4 checks passed
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.

2 participants