Uh oh!
There was an error while loading. Please reload this page.
[SPARK-49232][INFRA][TESTS] Remove 3.4.2 from SKIP_SPARK_RELEASE_VERSIONS - #47748
[SPARK-49232][INFRA][TESTS] Remove 3.4.2 from SKIP_SPARK_RELEASE_VERSIONS#47748panbingkun wants to merge 2 commits into
3.4.2 from SKIP_SPARK_RELEASE_VERSIONS#47748Conversation
panbingkun
commented
Aug 14, 2024
| .filterNot(_.contains("preview")) | ||
| .map("""<a href="spark-(\d.\d.\d)/">""".r.findFirstMatchIn(_).get.group(1)) | ||
| .filter(_ < org.apache.spark.SPARK_VERSION) | ||
| .filterNot(skipReleaseVersions.contains).toImmutableArraySeq |
There was a problem hiding this comment.
Maybe we can keep this function. I am not sure if there will be similar needs in the future that allow us to reuse it again.
There was a problem hiding this comment.
We should update pr title and pr description to match the change
dongjoon-hyun
left a comment
There was a problem hiding this comment.
Thank you for revision according to @LuciferYang 's comment.
Could you revise the PR title and description as suggested, @panbingkun ?
SKIP_SPARK_RELEASE_VERSIONS3.4.2 from SKIP_SPARK_RELEASE_VERSIONSpanbingkun
commented
Aug 14, 2024
Oh, I'm sorry, I miss to synchronize this update (Jira's title has also been updated). |
dongjoon-hyun
commented
Aug 14, 2024
Merged to master~ |
What changes were proposed in this pull request?
The pr aims to remove
3.4.2fromSKIP_SPARK_RELEASE_VERSIONS(build_maven.yml).Why are the changes needed?
Why was
SKIP_SPARK_RELEASE_VERSIONSintroduced? See here: https://issues.apache.org/jira/browse/SPARK-46400SPARK-46400 has already merged into Spark
3.4.3, and the corresponding issue no longer exists for Spark3.4.3+The following location only includes
the versionwe havefixedfor this issuehttps://dist.apache.org/repos/dist/release/spark/
Does this PR introduce any user-facing change?
No, only for GA's tests.
How was this patch tested?
Pass GA.
Was this patch authored or co-authored using generative AI tooling?
No.