Yicong-Huang opened a new pull request, #5689: URL: https://github.com/apache/texera/pull/5689
### What changes were proposed in this PR? `praw==7.6.1` was pinned in two requirements files: | File | Role | Status | | --- | --- | --- | | `amber/requirements.txt` | engine / SDK deps | **drop** the duplicate `praw` line | | `amber/operator-requirements.txt` | operator-specific deps (where praw belongs — only the Reddit Search operator uses it) | unchanged | Both files are installed sequentially by `bin/computing-unit-master.dockerfile` and `bin/computing-unit-worker.dockerfile`, so removing the duplicate is a runtime no-op — pip resolves the single remaining pin from `operator-requirements.txt`. Lockfile (`amber/system-requirements-lock.txt`) and `LICENSE-binary-python` are untouched because praw is still pulled in by the operator file. Diff: `1 file changed, 1 deletion(-)`. ### Any related issues, documentation, discussions? Closes #5687 (Supersedes #5688, which was wrongly scoped — that PR proposed to also delete the Reddit Search operator itself. Closed without merging.) ### How was this PR tested? - `git diff upstream/main` — confirms only the single `praw==7.6.1` line is removed from `amber/requirements.txt` - `grep -rn "praw" amber/` — confirms praw remains pinned in `operator-requirements.txt`, `system-requirements-lock.txt`, and `LICENSE-binary-python` - No source code uses praw directly (only the Reddit Search operator's generated Python in `RedditSearchSourceOpDesc.scala`), so the operator continues to work as before with the single remaining pin ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Claude Code (Opus 4.7 [1M context]) -- 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]
