andygrove opened a new pull request, #3251:
URL: https://github.com/apache/datafusion-comet/pull/3251
## Summary
This PR addresses the GitHub Actions cache storage approaching its 10GB
limit (was at 9.33 GB).
**Changes:**
1. **Cache optimization (saves ~2+ GB)**
- Remove Java version from cargo cache key (Rust target is
JDK-independent, was creating duplicate 2.1GB caches)
- Use `actions/cache/restore` + `actions/cache/save` pattern
- Only save cache on main branch, not on PRs (PRs restore from main's
cache)
2. **Reduce Rust test matrix**
- Consolidate from 2 jobs (Java 11 + Java 17) to 1 job (Java 17)
- Rust code is JDK-independent, so no coverage lost
3. **Add fast lint gate (~30 seconds)**
- New `lint` job runs `cargo fmt --check` before expensive builds
- `build-native` and `linux-test-rust` depend on lint passing
- Fail fast on formatting errors instead of waiting 5-10 minutes
- macOS lint runs on ubuntu-latest for cost efficiency
**Expected impact:**
- Cache usage: ~9.3 GB → ~5-6 GB (estimated)
- Time to fail on format error: 5-10 min → ~30 seconds
- Rust test jobs: 2 → 1
## Test plan
- [ ] Verify lint job runs first and gates other jobs
- [ ] Verify cache is only saved on main branch pushes
- [ ] Verify PR builds still work (restore from main's cache)
🤖 Generated with [Claude Code](https://claude.ai/code)
--
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]
