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

Merge JDBC Trino TPC jobs and Style Check jobs in GitHub Actions workflow #6170

Closed
wants to merge 3 commits into from

Conversation

pan3793
Copy link
Member

@pan3793 pan3793 commented Mar 12, 2024

🔍 Description

We are asked by ASF Infra to limit the max-parallel up to 10 for each workflow.

Merge JDBC Trino TPC jobs in GitHub Actions CI to reduce concurrency of workflow, also increase max-parallel of Spark jobs from 1 to 3

Also merge "shell check" job into "style check" job

Types of changes 🔖

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Test Plan 🧪

Pass GA.


Checklist 📝

Be nice. Be informative.

@github-actions github-actions bot added the kind:infra license, community building, project builds, asf infra related, etc. label Mar 12, 2024
@pan3793 pan3793 changed the title Merge JDBC Trino TPC jobs in GitHub Actions CI Merge JDBC Trino TPC jobs in GitHub Actions workflow Mar 12, 2024
@pan3793 pan3793 self-assigned this Mar 12, 2024
@pan3793 pan3793 added this to the v1.9.0 milestone Mar 12, 2024
@yaooqinn
Copy link
Member

Not related to this PR

But why we have 2 style check? https://github.com/apache/kyuubi/actions/runs/8244843719/job/22547751407?pr=6170

@pan3793
Copy link
Member Author

pan3793 commented Mar 12, 2024

Not related to this PR

But why we have 2 style check? https://github.com/apache/kyuubi/actions/runs/8244843719/job/22547751407?pr=6170

this looks can be merged too.

@pan3793 pan3793 changed the title Merge JDBC Trino TPC jobs in GitHub Actions workflow Merge JDBC Trino TPC jobs and Style Check jobs in GitHub Actions workflow Mar 12, 2024
@pan3793 pan3793 closed this in da2f079 Mar 12, 2024
@pan3793
Copy link
Member Author

pan3793 commented Mar 12, 2024

Thanks, merged to master

@pan3793 pan3793 deleted the ci-2 branch March 12, 2024 07:23
pan3793 added a commit that referenced this pull request Mar 13, 2024
…ub Actions workflow

We are asked by ASF Infra to limit the max-parallel up to 10 for each workflow.

Merge JDBC Trino TPC jobs in GitHub Actions CI to reduce concurrency of workflow, also increase max-parallel of Spark jobs from 1 to 3

Also merge "shell check" job into "style check" job

- [ ] Bugfix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to change)

Pass GA.

---

- [x] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html)

**Be nice. Be informative.**

Closes #6170 from pan3793/ci-2.

Closes #6170

9bf5053 [Cheng Pan] merge shell check
d0e7ece [Cheng Pan] nit
b171695 [Cheng Pan] Tune CI

Authored-by: Cheng Pan <[email protected]>
Signed-off-by: Cheng Pan <[email protected]>
zhaohehuhu pushed a commit to zhaohehuhu/incubator-kyuubi that referenced this pull request Mar 21, 2024
…n GitHub Actions workflow

# 🔍 Description

We are asked by ASF Infra to limit the max-parallel up to 10 for each workflow.

Merge JDBC Trino TPC jobs in GitHub Actions CI to reduce concurrency of workflow, also increase max-parallel of Spark jobs from 1 to 3

Also merge "shell check" job into "style check" job

## Types of changes 🔖

- [ ] Bugfix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to change)

## Test Plan 🧪

Pass GA.

---

# Checklist 📝

- [x] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html)

**Be nice. Be informative.**

Closes apache#6170 from pan3793/ci-2.

Closes apache#6170

9bf5053 [Cheng Pan] merge shell check
d0e7ece [Cheng Pan] nit
b171695 [Cheng Pan] Tune CI

Authored-by: Cheng Pan <[email protected]>
Signed-off-by: Cheng Pan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:infra license, community building, project builds, asf infra related, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants