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

Head commit for run:
45fc47577bc4908b23c3d674d91458fde7859f9d / Xinyuan Lin <[email protected]>
refactor(scheduling): move reuse decision to storage layer, flag to OutputPort

Address @Yicong-Huang's review on apache/texera#5707:
- Move the create-or-reuse decision out of RegionExecutionCoordinator into
  DocumentFactory.createOrReuseDocument -- it is storage-layer logic, not
  scheduling.
- Move the reuse flag off PhysicalOp onto the OutputPort proto, alongside the
  existing per-port `blocking`/`mode`; storage behavior is port-specific. The
  coordinator now reads it per output port and maintains no reuse state itself.
- Relocate the unit test to DocumentFactorySpec.

Per-port differentiation is still required (answering the "why not reuse for
all?" question): the loop back-edge re-executes LoopStart and every loop-body
operator on the same event as LoopEnd, but only LoopEnd accumulates -- the
others must recreate a fresh document each iteration.

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

With regards,
GitHub Actions via GitBox

Reply via email to