Hacktoberfest 2026：维护者为十月标记出来的 issue，仍然开放、适合新手。 浏览 Hacktoberfest issue

fix(grid): every result grid leaks (extension-method listener on PiiMaskingController is never removed); ~5 MB per run

未关闭
  • 哪里不对？
#1,555 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 1 天内回复

还没有人认领这个 Issue。

评估

难度
3/5
预计耗时
1-2 天
新手友好度
55/100
Issue 类型
缺陷
描述清晰度
描述清楚
活跃度
活跃
技术栈
dart, flutter
领域
desktop, performance

调研方向

Start with the listener registration and disposal in lib/features/workspace/result_grid_view.dart, then read the extension handlers in lib/features/workspace/grid/grid_pii_masking.dart and inspect the horizontal scroll listener pair. Add a guard under test/guards/ and a widget test that disposes VirtualResultGrid and checks PiiMaskingController.instance.hasListeners; run those tests. Done means listeners are removed and the guard catches extension method tear-offs used in listener pairs.

由索引模型根据 Issue 内容生成。

描述

bug data-grid P1 performance stability
Problem

Every result grid that is ever shown is kept in memory for the rest of the session, together with its whole result set and its widget / render / semantics tree. Each query run and each closed tab leaks one grid.

Cause: _VirtualResultGridState registers _onPiiMaskingChanged as a listener on the app-wide singleton PiiMaskingController.instance (lib/features/workspace/result_grid_view.dart:167) and removes it in dispose() (:392), but _onPiiMaskingChanged is declared in an extension _GridPiiMasking on _VirtualResultGridState (lib/features/workspace/grid/grid_pii_masking.dart:34). In Dart, tearing off an extension method creates a new closure each time and two such closures are not equal, so removeListener(_onPiiMaskingChanged) does not find the one that addListener(_onPiiMaskingChanged) stored. The listener stays in the singleton's list and keeps the state alive forever: state -> widget -> CompactResultDataset / rows -> element tree -> render objects -> semantics.

Minimal reproduction of the language behavior:

class Owner { void inClass() {} void reg(Notifier n){ n.add(inClass); n.add(ext);} void unreg(Notifier n){ n.remove(inClass); n.remove(ext);} }
extension E on Owner { void ext() {} }
// after reg + unreg: 1 listener is left (the extension tear-off); `o.inClass == o.inClass` is true, `o.ext == o.ext` is false
Measurements

Profile build of dev 0.4.22+5, app benchmark harness, Dart heap after a forced full GC (getAllocationProfile(gc: true)), heavySql = 5000 rows x 20 columns, retaining paths from the VM service:

Re-running a query 6 more times in the same tab

live _VirtualResultGridState live CompactResultDataset Dart heap
after 1 result 1 1 62 MB
after 6 re-runs 6 6 92.5 MB (+5 MB per run)

Opening a tab, running the query, closing the tab (confirming "Discard & Close"), 4 times

baseline after 4 cycles
mounted QueryEditorTab / ResultsTab 1 1
live QueryEditorTab instances 2 6
live ResultsTab 2 6
live VirtualResultGrid / _VirtualResultGridState 1 5
live CompactResultDataset 1 5
live FocusNode 105 235
live AnimationController 147 323
Dart heap 58 MB 87-90 MB

Retaining path of every leaked grid state (identical for all, 6 elements):

_VirtualResultGridState <- Context.0[] <- _Closure <- _List.N[] <- PiiMaskingController._listeners <- (static field PiiMaskingController.instance)

and of the leaked CompactResultDataset: _LazyDatasetRowList.dataset <- ResultsTab.rows <- _ResultsTabState._widget ... <- _VirtualResultGridState._focusNode ... <- PiiMaskingController._listeners.

A session of 100 queries on a 5000 x 20 result leaks on the order of 500 MB of Dart heap, more for larger results and a lot more for a result with a raised row cap.

Solution
  • Declare _onPiiMaskingChanged (and _onHorizontalScroll, which has the same shape) inside the State class instead of an extension, or keep the listener in a field: late final VoidCallback _piiListener = _onPiiMaskingChanged; and use that field for both addListener and removeListener.
  • A scan of lib/ found exactly these two addListener / removeListener pairs whose handler is an extension method; _onHorizontalScroll is attached to a controller that dispose() discards, so it is harmless in practice, but should be fixed the same way.
  • Add a guard test in test/guards/ that fails when the argument of addListener( / removeListener( is a method declared in an extension (the regex scan used for this report is enough).
Acceptance
  • PiiMaskingController.instance has no listeners after a VirtualResultGrid is disposed (widget test: pump the grid, remove it, assert hasListeners == false).
  • 6 re-runs of the same query in one tab leave 1 live _VirtualResultGridState (check with a leak test or leak_tracker).
  • Closing a tab releases QueryEditorTab, ResultsTab, the grid and its dataset.
  • Heap after N re-runs is flat in the app benchmark (add a leak_rerun scenario that prints live-instance counts after GC).

Related: #1549, #1552 (memory of the result path).

主要语言
Dart
星标
23
派生
1
平均合并
31 分钟
30 天内合并 PR
570

环境准备

在 Codespaces 中打开

在浏览器里用你自己的 GitHub 账号启动这个项目的开发容器。

从这里开始

  1. 先读完整个 Issue，再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库，在一个分支上完成修改。
  4. 提交 Pull Request，并在描述里引用这个 Issue 编号。

QueryaHub/Querya-Desktop 的其他 Issue

查看 QueryaHub/Querya-Desktop 的全部 Issue

相似的 Issue

更多 Dart Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。