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

[build] Optimize github actions structure and execution #5134

Merged
merged 1 commit into from
Mar 3, 2025

Conversation

liyubin117
Copy link
Contributor

@liyubin117 liyubin117 commented Feb 21, 2025

Purpose

  • Using strategy.matrix to unify highly same actions such as e2e-tests-flink-x to improve maintainlity, so we could just modify little lines to cover new version. In addition, the feature could parallelize executions, This allows faster execution, earlier occured errors and earlier released resources, also makes it easier to realize which part is causing the problem, increasing the efficiency of continuous delivery.
  • update github action components to latest version: actions/checkout@v4, actions/setup-java@v4, actions/cache@v4, peaceiris/actions-hugo@v3.
  • increase non-e2e testcases maven concurrency: 1 -> 2.
  • only compile the necessary modules.
  • update java distribution from adopt to temurin refer to url

Tests

API and Format

Documentation

@liyubin117 liyubin117 force-pushed the github-action-optimize branch 4 times, most recently from 9a2ddd0 to af241e4 Compare February 25, 2025 01:57
@liyubin117 liyubin117 force-pushed the github-action-optimize branch from af241e4 to e2f7c74 Compare February 27, 2025 09:05
@liyubin117
Copy link
Contributor Author

@JingsongLi CI passed and took less time, PTAL, thanks!

Copy link
Contributor

@JingsongLi JingsongLi left a comment

Choose a reason for hiding this comment

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

+1

@JingsongLi JingsongLi merged commit e95b4cf into apache:master Mar 3, 2025
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants