Skip to content

Conversation

@dongjoon-hyun
Copy link
Member

@dongjoon-hyun dongjoon-hyun commented Oct 15, 2025

What changes were proposed in this pull request?

This PR allows users to use {{SPARK_VERSION}} in the following three configs.

spark.kubernetes.container.image
spark.kubernetes.driver.container.image
spark.kubernetes.executor.container.image

Why are the changes needed?

We should implement this in Apache Spark K8s Operator for feature parity with Apache Spark 4.1.0. Otherwise, it will be replaced with the compile dependency, 4.1.0-preview2 instead of the value of RuntimeVersions.sparkVersion.

Does this PR introduce any user-facing change?

No because {{SPARK_VERSION}} is a newly supported pattern.

How was this patch tested?

Pass the CIs with newly added test cases.

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

No.

@dongjoon-hyun
Copy link
Member Author

cc @peter-toth

Copy link
Member

@viirya viirya left a comment

Choose a reason for hiding this comment

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

Do we need to add some examples for this?

@dongjoon-hyun
Copy link
Member Author

Thank you. We can add two examples for both SparkApplication and SparkCluster. So, I didn't add them here.

@dongjoon-hyun
Copy link
Member Author

Merged to main.

Copy link
Contributor

@peter-toth peter-toth left a comment

Choose a reason for hiding this comment

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

Late LGTM.

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.

3 participants