You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@texera.apache.org by "bobbai00 (via GitHub)" <gi...@apache.org> on 2025/12/08 18:58:51 UTC

[PR] chore: drop R support flags from Dockerfiles and CI workflow [texera]

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: notifications-unsubscribe@texera.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] chore: drop R support flags from Dockerfiles and CI workflow [texera]

Posted by "bobbai00 (via GitHub)" <gi...@apache.org>.
bobbai00 merged PR #4124:
URL: https://github.com/apache/texera/pull/4124


-- 
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: notifications-unsubscribe@texera.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org