Skip to content
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

Upgrade to Spark 3.5.4 #2379

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

jacobsalway
Copy link
Member

@jacobsalway jacobsalway commented Jan 9, 2025

Purpose of this PR

Upgrade to Spark 3.5.4

Change Category

  • Bugfix (non-breaking change which fixes an issue)
  • Feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that could affect existing functionality)
  • Documentation update

Rationale

Checklist

  • I have conducted a self-review of my own code.
  • I have updated documentation accordingly.
  • I have added tests that prove my changes are effective or that my feature works.
  • Existing unit tests pass locally with my changes.

Additional Notes

Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please ask for approval from jacobsalway. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Signed-off-by: Jacob Salway <jacob.salway@gmail.com>
@@ -77,6 +77,7 @@ The following table lists the most recent few versions of the operator.

| Operator Version | API Version | Kubernetes Version | Base Spark Version |
|-----------------------|-------------|--------------------|--------------------|
| `v2.1.0` | `v1beta2` | 1.16+ | `3.5.3` |
Copy link
Member Author

Choose a reason for hiding this comment

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

We can add another row for 3.5.4 later depending on whether this ends up in a minor or patch release.

Copy link
Contributor

Choose a reason for hiding this comment

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

What about bumping the Spark version to v3.5.4 when we are going to release the next minor version i.e. v2.2.0?

Copy link
Member Author

Choose a reason for hiding this comment

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

Sure, I'll put this back into draft and mark it as frozen so we can take it up in the next minor release.

@jacobsalway jacobsalway marked this pull request as ready for review January 9, 2025 02:17
@google-oss-prow google-oss-prow bot requested a review from mwielgus January 9, 2025 02:17
@jacobsalway
Copy link
Member Author

/assign @ChenYi015

@jacobsalway
Copy link
Member Author

/lifecycle frozen

Holding for next minor release.

Copy link
Contributor

@jacobsalway: The lifecycle/frozen label cannot be applied to Pull Requests.

In response to this:

/lifecycle frozen

Holding for next minor release.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@jacobsalway
Copy link
Member Author

/hold for next minor release.

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