The GitHub Actions job "Tests (AMD)" on airflow.git/backport-93d722e-v3-3-test 
has succeeded.
Run started by GitHub user github-actions[bot] (triggered by pierrejeambrun).

Head commit for run:
e9505bb08890e7de96a9428737f6bb22dbcc4ce6 / Pierre Jeambrun 
<[email protected]>
[v3-3-test] Re-render only the changed column when Grid summaries stream in 
(#69917)

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).
(cherry picked from commit 93d722e9b82faca703e34ea0e5461fbd4689bdaf)

Co-authored-by: Pierre Jeambrun <[email protected]>
related: #69531

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

With regards,
GitHub Actions via GitBox


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

Reply via email to