Skip to content

ci: Use install-action instead of cargo install to speed up CI#23477

Open
2010YOUY01 wants to merge 1 commit into
apache:mainfrom
2010YOUY01:ci-install-action
Open

ci: Use install-action instead of cargo install to speed up CI#23477
2010YOUY01 wants to merge 1 commit into
apache:mainfrom
2010YOUY01:ci-install-action

Conversation

@2010YOUY01

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

  • Closes #.

Rationale for this change

For CI dependencies installed with cargo install, it's possible to use install-action instead for faster setup. (former compiles the binary, the latter directly download the release version) , each of them should get ~1min faster.

What changes are included in this PR?

  • Use install-action instead of cargo install for existing CI jobs
  • Add a new CI job to enforce this convention (grep cargo install for all Github Action scripts)

Are these changes tested?

Are there any user-facing changes?

@github-actions github-actions Bot added the development-process Related to development process of DataFusion label Jul 11, 2026
@2010YOUY01 2010YOUY01 marked this pull request as draft July 11, 2026 13:41
@2010YOUY01

Copy link
Copy Markdown
Contributor Author

The step of installing cargo machete improved from 41s to 3s

https://github.com/apache/datafusion/actions/runs/29175818974/job/86604744160
https://github.com/apache/datafusion/actions/runs/29154744229/job/86549929396?pr=23477

Other cargo dependencies should have a similar speedup. Note some of them are not faster, because install-action don't support all cargo dependencies, unsupported ones will fallback to cargo install. It seems all of them can be manually added by sending a PR to install-action

See https://github.com/taiki-e/install-action/blob/main/TOOLS.md

@2010YOUY01 2010YOUY01 marked this pull request as ready for review July 12, 2026 02:09

@comphead comphead left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks @2010YOUY01 lets see this in practice, hope it would make easier with retries and https issues during download

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

development-process Related to development process of DataFusion

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants