mirror of
https://github.com/apache/superset.git
synced 2026-05-12 19:35:17 +00:00
08aac73101
The former shouldComponentUpdate was re-implemented as a useMemo whose return value was ignored, which is a misuse of useMemo (it was being used purely for ref mutation side effects). Since the component is already wrapped in memo(), the shallow prop compare handles re-render gating; the custom matrixify / shouldRender logic was dead code. Replace with: - useMemo to clone queriesResponse (legitimate memoization) - A prev-ref + useEffect pattern to track queriesResponse changes for render callbacks (safe to read refs during render when the value doesn't influence render output) Drop the now-unused PrevPropsRef interface and the dead matrixify comparator.