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

[SPARK-48174][INFRA] Merge connect back to the original test pipeline #46441

Closed
wants to merge 1 commit into from

Conversation

dongjoon-hyun
Copy link
Member

@dongjoon-hyun dongjoon-hyun commented May 7, 2024

What changes were proposed in this pull request?

This PR aims to merge connect back to the original test pipeline to reduce the maximum concurrency of GitHub Action by one.

Why are the changes needed?

This is a partial recover from the following.

We stabilized the root cause of #45107 via the following PRs. In addition we will disable a flaky test case if exists.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Pass the CIs.

Was this patch authored or co-authored using generative AI tooling?

No.

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

Could you review this INFRA PR, @cloud-fan ?

@dongjoon-hyun
Copy link
Member Author

Could you review this PR, @viirya ?

@viirya
Copy link
Member

viirya commented May 7, 2024

Looks good to me.

@dongjoon-hyun
Copy link
Member Author

Thank you, @viirya !
Merged to master.

@dongjoon-hyun dongjoon-hyun deleted the SPARK-48174 branch May 7, 2024 16:35
@cloud-fan
Copy link
Contributor

late LGTM!

@dongjoon-hyun
Copy link
Member Author

Thank you, @cloud-fan !

JacobZheng0927 pushed a commit to JacobZheng0927/spark that referenced this pull request May 11, 2024
### What changes were proposed in this pull request?

This PR aims to merge connect back to the original test pipeline to reduce the maximum concurrency of GitHub Action by one.
- https://infra.apache.org/github-actions-policy.html
  > All workflows SHOULD have a job concurrency level less than or equal to 15.

### Why are the changes needed?

This is a partial recover from the following.
- apache#45107

We stabilized the root cause of apache#45107 via the following PRs. In addition we will disable a flaky test case if exists.

- apache#46395
- apache#46396
- apache#46425

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

No.

### How was this patch tested?

Pass the CIs.

### Was this patch authored or co-authored using generative AI tooling?

No.

Closes apache#46441 from dongjoon-hyun/SPARK-48174.

Authored-by: Dongjoon Hyun <dhyun@apple.com>
Signed-off-by: Dongjoon Hyun <dhyun@apple.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants