Skip to content

Spark upgrade version 3.5.5 and 4.1.0 - #768

Merged
chloe41427[bot] merged 5 commits into
sdkman:masterfrom
aganisatria:spark-upgrade-version-3.5.5-and-4.2.0-preview4
Jun 3, 2026
Merged

Spark upgrade version 3.5.5 and 4.1.0#768
chloe41427[bot] merged 5 commits into
sdkman:masterfrom
aganisatria:spark-upgrade-version-3.5.5-and-4.2.0-preview4

Conversation

@aganisatria

Copy link
Copy Markdown
Contributor

Upgraded spark 3.5.5 and 4.2.0-preview4

@aganisatria

Copy link
Copy Markdown
Contributor Author

Hi sir @marc0der, this PR is ready for review. Could you please take a look and also help approve the workflows?
Thank you!

@aganisatria aganisatria reopened this Apr 14, 2026
@marc0der

Copy link
Copy Markdown
Member

@aganisatria This PR should be additive to what's already in the database, but it looks like you are re-inserting items that are already there. Please make the versions added mutually exclusive to those of the previous migration.

@aganisatria

Copy link
Copy Markdown
Contributor Author

Noted, will do sir @marc0der

@aganisatria

Copy link
Copy Markdown
Contributor Author

Done sir @marc0der.
Since this is my first PR, just let me know if there is required step that i need to do.

@marc0der marc0der left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looking good. Just some minor improvements to make it better.

Comment thread src/main/scala/io/sdkman/changelogs/SparkMigrations.scala Outdated
Comment thread src/main/scala/io/sdkman/changelogs/SparkMigrations.scala Outdated
@aganisatria

Copy link
Copy Markdown
Contributor Author

Done updated sir @marc0der, looking forward to this update to help maintain EMR 7.10.0 ☺️

@aganisatria

Copy link
Copy Markdown
Contributor Author

Hi sir @marc0der, apologize to disturb your time.
bump for this please thank you so much

@chloe41427

chloe41427 Bot commented May 15, 2026

Copy link
Copy Markdown
Contributor

Thanks for the contribution @aganisatria! One request before we merge: please drop the 4.2.0-preview4 entry. Preview releases tend to have unreliable archive URLs that break shortly after the stable release ships. We prefer to keep migrations to stable releases only.

Once that's removed, this looks good to go! 🙂

@aganisatria aganisatria changed the title Spark upgrade version 3.5.5 and 4.2.0-preview4 Spark upgrade version 3.5.5 and 4.1.0 Jun 2, 2026
@marc0der

marc0der commented Jun 2, 2026

Copy link
Copy Markdown
Member

@aganisatria scalafmt failed

@aganisatria

aganisatria commented Jun 3, 2026

Copy link
Copy Markdown
Contributor Author

done sir @marc0der, fixed.

@chloe41427 chloe41427 Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good — build passes, scalafmt is fixed. Thanks for iterating on this! 🎉

@chloe41427
chloe41427 Bot merged commit c46f7f2 into sdkman:master Jun 3, 2026
1 check 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