Skip to content

[Fix] Support for Spark versions with multiple Scala versions#1345

Merged
rauchy merged 5 commits intomainfrom
tf-issue-5218-clean
Dec 3, 2025
Merged

[Fix] Support for Spark versions with multiple Scala versions#1345
rauchy merged 5 commits intomainfrom
tf-issue-5218-clean

Conversation

@rauchy
Copy link
Copy Markdown
Contributor

@rauchy rauchy commented Dec 2, 2025

What changes are proposed in this pull request?

This is a fix for databricks/terraform-provider-databricks#5218. The issue is cased by the fact that Scala selector had default value of 2.12 (for Terraform only), and as result was filtering out 17.x versions that are released only with Scala 2.13.

Fix is to relax the default value, but handle the case when there could be 2.12 and 2.13 versions for the same DBR version (16.x only).

Removed all Terraform customizations from the Go SDK struct, will be handled separately in Terraform itself

Also bumped staticcheck to 0.6.1 to support compilation with Go 1.25

How is this tested?

Added unit test, tested manually

This is a fix for databricks/terraform-provider-databricks#5218.
The issue is cased by the fact that `Scala` selector had default value of `2.12` (for
Terraform only), and as result was filtering out 17.x versions that are released only with
Scala 2.13.

Fix is to relax the default value, but handle the case when there could be 2.12 and 2.13
versions for the same DBR version (16.x only).

Also bumped staticcheck to 0.6.1 to support compilation with Go 1.25

Signed-off-by: Alex Ott <[email protected]>
@rauchy rauchy temporarily deployed to test-trigger-is December 2, 2025 16:05 — with GitHub Actions Inactive
@github-actions
Copy link
Copy Markdown

github-actions bot commented Dec 2, 2025

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/sdk-go

Inputs:

  • PR number: 1345
  • Commit SHA: 0fc2252e021c820da46542eccde69b7a4057794e

Checks will be approved automatically on success.

@rauchy rauchy temporarily deployed to test-trigger-is December 2, 2025 16:06 — with GitHub Actions Inactive
@rauchy rauchy requested a review from tanmay-db December 2, 2025 16:07
@rauchy rauchy added this pull request to the merge queue Dec 3, 2025
Merged via the queue into main with commit 3276057 Dec 3, 2025
15 checks passed
@rauchy rauchy deleted the tf-issue-5218-clean branch December 3, 2025 08:10
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.

3 participants