Yicong-Huang opened a new pull request, #4636:
URL: https://github.com/apache/texera/pull/4636
### What changes were proposed in this PR?
Restore `uv pip install --system` in `build.yml`'s scala and python jobs.
The faster install path landed in #4521 was unintentionally rewritten back to
stock `pip install` while inlining binary license checks in #4597, and the
regression has been carried forward by every subsequent rebase.
Diff per job (scala and python both reset to the post-#4521 form):
```yaml
- name: Install dependencies
run: |
python -m pip install uv
if [ -f amber/requirements.txt ]; then uv pip install --system -r
amber/requirements.txt; fi
if [ -f amber/operator-requirements.txt ]; then uv pip install --system
-r amber/operator-requirements.txt; fi
# python job only:
if [ "${{ matrix.python-version }}" = "3.12" ]; then uv pip install
--system pip-licenses; fi
```
No behaviour change beyond install speed.
### Any related issues, documentation, discussions?
Closes #4635. Restores #4521. Regression introduced by #4597.
### How was this PR tested?
Will be exercised by this PR's own scala and python matrices once CI runs.
The same `uv pip` invocation is what those jobs used to run between #4521 and
#4597 without issues.
### Was this PR authored or co-authored using generative AI tooling?
Generated-by: Claude Opus 4.7
--
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]
