The GitHub Actions job "Tests (AMD)" on airflow.git/grid-stream-cascade has 
failed.
Run started by GitHub user pierrejeambrun (triggered by pierrejeambrun).

Head commit for run:
d10a9ee4338f7c333a37527ff0f27e0eb0e8ef35 / pierrejeambrun 
<[email protected]>
Re-render only the changed column when Grid summaries stream in

The Dag Grid streams TI summaries as one NDJSON line per run and stores them in
a Map; every line replaced the Map and re-rendered Grid. Because useVirtualizer
(@tanstack/react-virtual) is on the React Compiler's known-incompatible list, 
the
compiler skips optimizing all of Grid — so flatNodes and the click handlers got
fresh references on every render and every column re-rendered on every line, not
just the run whose summary arrived. On a ~20-run x 30-task grid that was 117
spurious column re-renders per load.

Memoize the values handed down to the columns by hand — the escape hatch the
compiler's own bailout message points to for incompatible-library APIs. Measured
on the same grid: column re-renders per load dropped 171 -> 54 (all legitimate),
grid load render work ~47% lower.

Also drop a redundant immediate stream restart: an unconditional refresh tick
fired the instant the interval effect mounted, aborting and reopening the
just-opened mount stream (an AbortError on every grid mount with active runs).

related: #69531

Report URL: https://github.com/apache/airflow/actions/runs/29424419414

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to