Skip to content

[SPARK-43845][INFRA] Setup Scala 2.12 Daily GitHub Action Job#41354

Closed
dongjoon-hyun wants to merge 1 commit intoapache:masterfrom
dongjoon-hyun:SPARK-43845
Closed

[SPARK-43845][INFRA] Setup Scala 2.12 Daily GitHub Action Job#41354
dongjoon-hyun wants to merge 1 commit intoapache:masterfrom
dongjoon-hyun:SPARK-43845

Conversation

@dongjoon-hyun
Copy link
Member

@dongjoon-hyun dongjoon-hyun commented May 29, 2023

What changes were proposed in this pull request?

This PR aims to setup Scala 2.12 Daily GitHub Action Job by converting Scala 2.13 Daily Job.

  • Rename build_scala213.yml file to .github/workflows/build_scala212.yml.
  • Rename job name
-name: "Build (master, Scala 2.13, Hadoop 3, JDK 8)"
+name: "Build (master, Scala 2.12, Hadoop 3, JDK 8)"
  • Switch SCALA_PROFILE from scala2.13 to scala2.12.

Why are the changes needed?

To keep the Scala 2.12 test coverage

Does this PR introduce any user-facing change?

No.

How was this patch tested?

This is a daily job. We need to check after merging.

@github-actions github-actions bot added the INFRA label May 29, 2023
@dongjoon-hyun
Copy link
Member Author

Could you review this too, @HyukjinKwon ?

@dongjoon-hyun
Copy link
Member Author

Thank you, @yaooqinn and @LuciferYang .
Merged to master.

@dongjoon-hyun dongjoon-hyun deleted the SPARK-43845 branch May 29, 2023 03:08
@HyukjinKwon
Copy link
Member

Should probably hold on making Scala 2.13 by default for now. it'd be a breaking change to the end users of using pip install pyspark, and they wouldn't have a workaround to avoid that.

czxm pushed a commit to czxm/spark that referenced this pull request Jun 12, 2023
### What changes were proposed in this pull request?

This PR aims to setup Scala 2.12 Daily GitHub Action Job by converting Scala 2.13 Daily Job.
- Rename `build_scala213.yml` file to `.github/workflows/build_scala212.yml`.
- Rename job name
```
-name: "Build (master, Scala 2.13, Hadoop 3, JDK 8)"
+name: "Build (master, Scala 2.12, Hadoop 3, JDK 8)"
```
- Switch SCALA_PROFILE from `scala2.13` to `scala2.12`.

### Why are the changes needed?

To keep the Scala 2.12 test coverage

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

This is a daily job. We need to check after merging.

Closes apache#41354 from dongjoon-hyun/SPARK-43845.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Comments