-
Notifications
You must be signed in to change notification settings - Fork 113
fix: pin R arrow package to version 14.0.2.1 in computing-unit-master #3761
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: pin R arrow package to version 14.0.2.1 in computing-unit-master #3761
Conversation
- Split R package installation into two steps - Install coro and dplyr packages first - Pin arrow package to version 14.0.2.1 using remotes::install_version - Add upgrade='never' to prevent automatic upgrades - Include verification of installed arrow version 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
|
@aglinxinyuan @mengw15 Please review this PR. |
|
I added @aglinxinyuan to the reviewer list as you are familiar with the overall structure of pinning libraries. |
aglinxinyuan
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
|
@chenlica @aglinxinyuan Arrow 14.0.2.1 make the R operators work. However, 14.0.2.1 seems too old compared to arrow's latest version (21.0.0). Can I let @mengw15 to investigate why 21.0.0 doesn't work for texera's R operators, and find a more recent version of R arrow that supports R UDF operators? |
mengw15
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Sure. Is this investigation blocking? |
It is not blocking the release. |
Summary
This PR pins the R arrow package to version 14.0.2.1 in the computing-unit-master Dockerfile to ensure compatibility with R operators.
Problem
The R arrow package was not pinned to a specific version, causing the latest version (21.0.0) to be installed. This version is incompatible with Texera's R operators, causing them to fail at runtime.
Solution
arrowpackage to version 14.0.2.1 usingremotes::install_version()upgrade='never'parameter to prevent automatic upgradesFixes #3760