The GitHub Actions job "Required Checks" on texera.git/xinyuan-loop-feb has 
failed.
Run started by GitHub user aglinxinyuan (triggered by aglinxinyuan).

Head commit for run:
3d4f15b4454fde2afd9bc6f60b133b6a1c606468 / Xinyuan Lin <[email protected]>
refactor(loop): rename isLoopEnd flag to reusesOutputStorageOnReExecution

Addresses PR #4206 review feedback
(https://github.com/apache/texera/pull/4206#discussion_r3400851514): the
PhysicalOp flag was named after the operator (isLoopEnd) rather than the
behavior the scheduler actually checks -- 'keep this operator's output
storage across a region re-run.' (This rename was made earlier but lost
when the branch was rebased.)

* PhysicalOp: isLoopEnd -> reusesOutputStorageOnReExecution (+ doc
  comment); withIsLoopEnd -> withReusesOutputStorageOnReExecution.
* RegionExecutionCoordinator: the skip-recreate guard checks
  reusesOutputStorageOnReExecution; local val + comment reworded to the
  behavior.
* LoopEndOpDesc sets .withReusesOutputStorageOnReExecution(true).
* Loop specs + mixin updated to the new name.

Now any future operator that must preserve its output across a region
re-execution can set the flag without a LoopEnd-specific misnomer.

Verified: WorkflowCore + WorkflowOperator compile and all 29 LoopStart/
LoopEnd op-desc specs pass. (The amber module's local compile is blocked
by an unrelated pre-existing JOOQ issue -- PveManager references the
virtual_environments table from #5577, absent from the un-migrated local
DB; CI compiles it against a fresh schema.)

Report URL: https://github.com/apache/texera/actions/runs/27450263859

With regards,
GitHub Actions via GitBox

Reply via email to