Suryansh-Dey opened a new pull request, #20186:
URL: https://github.com/apache/datafusion/pull/20186

   ## Which issue does this PR close?
   
   None. An improvement.
   
   ## Rationale for this change
   
   Several GitHub Actions workflows were missing `rust-cache`, causing 
redundant recompilation of dependencies and tool installations on every run. 
This increases CI time and resource usage unnecessarily.
   
   ## What changes are included in this PR?
   
   Added `Swatinem/rust-cache` to the following workflows:
   
   | Workflow | Jobs Modified | What's Cached |
   |----------|---------------|---------------|
   | 
[dev.yml](cci:7://file:///home/suryansh/dev/GSOC/datafusion/.github/workflows/dev.yml:0:0-0:0)
 | `license-header-check`, `typos` | `hawkeye`, `typos-cli` tool installations |
   | 
[docs.yaml](cci:7://file:///home/suryansh/dev/GSOC/datafusion/.github/workflows/docs.yaml:0:0-0:0)
 | `build-docs` | `cargo-depgraph` installation |
   | 
[docs_pr.yaml](cci:7://file:///home/suryansh/dev/GSOC/datafusion/.github/workflows/docs_pr.yaml:0:0-0:0)
 | `linux-test-doc-build` | `cargo-depgraph` installation |
   | 
[extended.yml](cci:7://file:///home/suryansh/dev/GSOC/datafusion/.github/workflows/extended.yml:0:0-0:0)
 | `linux-build-lib`, `linux-test-extended`, `hash-collisions`, 
`sqllogictest-sqlite` | Build artifacts and dependencies |
   
   **Notes:**
   - Existing `cargo clean` steps in 
[extended.yml](cci:7://file:///home/suryansh/dev/GSOC/datafusion/.github/workflows/extended.yml:0:0-0:0)
 are preserved to prevent disk space exhaustion on standard GitHub runners
   - Cache is only saved on `main` branch to avoid polluting the cache with 
PR-specific builds
   - Used `cache-targets: false` for tool-only caching jobs to minimize cache 
size
   
   ## Are these changes tested?
   
   - Validated YAML syntax of all modified files
   - Changes follow existing patterns in 
[rust.yml](cci:7://file:///home/suryansh/dev/GSOC/datafusion/.github/workflows/rust.yml:0:0-0:0)
 which already uses `rust-cache`
   
   ## Are there any user-facing changes?
   
   No - this is a CI-only improvement that should reduce workflow execution 
time.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to