bobbai00 opened a new pull request, #3929: URL: https://github.com/apache/texera/pull/3929
## Summary - Add `WITH_R_SUPPORT` build argument to both `computing-unit-master.dockerfile` and `computing-unit-worker.dockerfile` - Conditionally install R dependencies based on the build flag - Defaults to `false` to reduce image size and build time when R support is not needed ## Related Issue Closes #3928 ## Changes ### computing-unit-master.dockerfile - Added `ARG WITH_R_SUPPORT=false` (default: false for backward compatibility) - Conditionally install R-related system packages (gfortran, libreadline, libxml2, etc.) - Conditionally compile and install R 4.3.3 - Conditionally install R packages (coro, dplyr, arrow) ### computing-unit-worker.dockerfile - Applied same conditional R support changes as master dockerfile - Maintains consistency across both computing unit images ## Impact **Without R support (default):** - Significantly smaller image size (~500MB+ reduction) - Faster build time (avoids 10+ minute R compilation) - Suitable for deployments that don't use R operators **With R support:** \`\`\`bash docker build --build-arg WITH_R_SUPPORT=true -f bin/computing-unit-master.dockerfile . \`\`\` ## Test Plan - [ ] Build images with and without R support flag - [ ] Verify R operators work when flag is enabled - [ ] Verify non-R workflows work when flag is disabled - [ ] Test in Kubernetes deployment 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]> -- 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]
