You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@texera.apache.org by GitBox <gi...@apache.org> on 2026/05/27 03:51:34 UTC
[GH] (texera/gh-readonly-queue/main/pr-5077-8a7366f3b1ec18fc118a92b76326c06b92be1ff3): Workflow run "Required Checks" is working again!
The GitHub Actions job "Required Checks" on texera.git/gh-readonly-queue/main/pr-5077-8a7366f3b1ec18fc118a92b76326c06b92be1ff3 has succeeded.
Run started by GitHub user Yicong-Huang (triggered by Yicong-Huang).
Head commit for run:
d5bc8b7cf8d2e515280934a9c3e52c305fe3c002 / Matthew B. <mg...@uci.edu>
refactor: untrack generated python proto, regenerate in build (#5077)
### What changes were proposed in this PR?
Stop version-tracking the betterproto-generated Python files under
`amber/src/main/python/proto/`. Add a `genPythonProto` sbt task that
runs `bin/python-proto-gen.sh` as a dependency of
`WorkflowExecutionService/Compile/compile`, with a graceful skip when
`protoc` is missing. Wire `protoc` 3.19.4 (matching `PB.protocVersion`)
and `betterproto[compiler]==2.0.0b7` into the three image build stages
(worker,master, web-app) and the three CI jobs that touch Python proto
(`python`, `python-state-materialization-mac`, `amber-integration`). The
14th tracked file (`proto/org/apache/texera/web/__init__.py`) had no
corresponding `.proto` source and is gone after regen; grep confirms no
callers.
### Any related issues, documentation, or discussions?
Closes: #4102
### How was this PR tested?
locally: sbt clean WorkflowExecutionService/compile regenerates the tree
and pytest -m "not integration" passes; skip path verified with protoc
absent. All three Docker images build end-to-end. CI covers the workflow
YAML on
push.
### Was this PR authored or co-authored using generative AI tooling?
Co-authored with Claude Opus 4.7 in compliance with ASF
Report URL: https://github.com/apache/texera/actions/runs/26489297441
With regards,
GitHub Actions via GitBox