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

Head commit for run:
0b55650d0986a13c8684ee0513194299b2e231b4 / Xinyuan Lin <[email protected]>
refactor(loop): dedup loop-operator eval/state helpers, drop redundant flag

A /simplify pass over the loop changes in operator.py:

- Extract `_strip_reserved(state)` -- the `{k: v ... if k not in
  _RESERVED_STATE_KEYS}` comprehension was copy-pasted three times
  (produce_state_on_finish + twice in run_update).
- Extract `_eval_loop_expr(expr, state, table)` -- LoopStart.eval_output and
  LoopEnd.eval_condition were near-identical "exec the expression as `output`
  in a throwaway namespace" evaluators.
- Drop LoopEndOperator._consumed_state: it was redundant with _loop_table,
  which is None exactly until run_update consumes a matching state. condition()
  now short-circuits on `self._loop_table is None`.

Behavior unchanged; test_loop_operators / test_main_loop / test_output_manager
stay green (the one unit test that pinned the flag now pins the equivalent
_loop_table state). Net fewer lines and no duplicated logic.

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

With regards,
GitHub Actions via GitBox

Reply via email to