bobbai00 opened a new pull request, #4124:
URL: https://github.com/apache/texera/pull/4124
### What changes were proposed in this PR?
This PR removes the `WITH_R_SUPPORT` build argument and all R-related
installation logic from the Docker build configuration:
1. **Dockerfiles** (`computing-unit-master.dockerfile` and
`computing-unit-worker.dockerfile`):
- Removed `ARG WITH_R_SUPPORT` build argument
- Removed conditional R runtime dependencies installation
- Removed R compilation and installation steps (R 4.3.3)
- Removed R packages installation (arrow, coro, dplyr)
- Removed `LD_LIBRARY_PATH` environment variable for R libraries
- Removed `r-requirements.txt` copy in worker dockerfile
- Simplified to Python-only dependencies
2. **GitHub Actions Workflow**
(`.github/workflows/build-and-push-images.yml`):
- Removed `with_r_support` workflow input parameter
- Removed `with_r_support` from job outputs and parameter passing
- Removed `WITH_R_SUPPORT` build args from both AMD64 and ARM64 build
steps
- Removed R Support from build summary
**Benefits:**
- Simplified Docker images (smaller size, faster builds)
- Reduced build complexity and maintenance burden
- Removed unused feature per issue #4090 (Runtime R support was dropped)
### Any related issues, documentation, discussions?
Related to #4090 - Drop Runtime Support for R-UDF
### How was this PR tested?
- Verified Dockerfile syntax is valid
- Confirmed all references to `WITH_R_SUPPORT` have been removed
- CI workflow changes preserve existing build functionality for Python-based
operators
- Local testing: Docker build commands work correctly without the R support
flag
### Was this PR authored or co-authored using generative AI tooling?
Generated-by: Claude Sonnet 4.5 (claude-sonnet-4-5-20250929) via Claude Code
CLI
--
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]
