Yicong-Huang opened a new pull request, #5688: URL: https://github.com/apache/texera/pull/5688
### What changes were proposed in this PR? Removes the `Reddit Search` source operator and the unmaintained `praw==7.6.1` / `prawcore==2.4.0` Python deps. Reddit Search was a thin PRAW wrapper that emitted Python code calling `praw.Reddit(...)` from inside `RedditSearchSourceOpDesc`; deleting it lets us drop a third-party API surface and two transitive license entries. | Layer | File | Action | | --- | --- | --- | | Operator descriptor | `common/workflow-operator/.../source/apis/reddit/RedditSearchSourceOpDesc.scala` | delete | | Operator enum | `common/workflow-operator/.../source/apis/reddit/RedditSourceOperatorFunction.java` | delete | | Operator registry | `common/workflow-operator/.../operator/LogicalOp.scala` | drop import + `@Type(... "RedditSearch")` | | Python deps | `amber/requirements.txt`, `amber/operator-requirements.txt`, `amber/system-requirements-lock.txt` | drop `praw`, `prawcore` | | License manifest | `amber/LICENSE-binary-python` | drop `praw`, `prawcore` rows | | Frontend asset | `frontend/src/assets/operator_images/RedditSearch.png` | delete | | Docs | `docs/reference/operators/external-api/reddit-search.md`, `docs/reference/operators/external-api/_index.md` | delete page; drop index row | | Contributing guide | `docs/contribution-guidelines/guide-to-implement-java-operator.md` | genericize sample `@JsonPropertyDescription` text (was Reddit-specific) | Result: `11 files changed, 2 insertions(+), 245 deletions(-)`. Frontend looks up operator PNGs by `operatorType` (`assets/operator_images/<type>.png`), so dropping the Type registration is sufficient — no other UI references to `RedditSearch` exist. Left in place intentionally: `update-checker` and `websocket-client` in `system-requirements-lock.txt`. They are praw transitives, but the lock file is regenerated separately and these may be pulled by other deps — keeping the lock file truthful pending regeneration. ### Any related issues, documentation, discussions? Closes #5687 ### How was this PR tested? - `STORAGE_JDBC_USERNAME=texera STORAGE_JDBC_PASSWORD=password sbt "WorkflowOperator/compile"` — clean (260 Scala / 49 Java sources, 21s) - `sbt scalafmtCheckAll` — clean - `grep -rni "reddit\|praw" --include="*.scala" --include="*.java" --include="*.py" --include="*.ts" --include="*.html" --include="*.json" --include="*.txt" --include="*.yaml" --include="*.yml" --include="*.md"` — no remaining references ### 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]
