Skip to content

Trace quick access table disposal#4083

Draft
trancexpress wants to merge 2 commits into
eclipse-platform:masterfrom
trancexpress:gh4009
Draft

Trace quick access table disposal#4083
trancexpress wants to merge 2 commits into
eclipse-platform:masterfrom
trancexpress:gh4009

Conversation

@trancexpress

@trancexpress trancexpress commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Tracing for fail in QuickAccessDialogTest hints at a disposed table prior to updating entries in the table. This change adds tracing for table dispose to validate this.

See: #4009

@github-actions

github-actions Bot commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Test Results

   861 files  ± 0     861 suites  ±0   45m 40s ⏱️ - 12m 28s
 8 034 tests ± 0   7 789 ✅  -  2  243 💤 ± 0  2 ❌ +2 
20 451 runs   - 90  19 817 ✅  - 69  632 💤  - 23  2 ❌ +2 

For more details on these failures, see this check.

Results for commit 23fb09d. ± Comparison against base commit 968d4ce.

♻️ This comment has been updated with latest results.

@trancexpress trancexpress force-pushed the gh4009 branch 3 times, most recently from 9844577 to 07adab6 Compare June 11, 2026 11:29
@trancexpress

trancexpress commented Jun 11, 2026

Copy link
Copy Markdown
Contributor Author

The tests in QuickAccessDialogTest seem to be failing a lot with this change... I'm not sure why. All the fails are on MacOS, similar to #4009...

Unfortunately the new tracing is hard to use, without knowing which test is running (still no clue where the workspace logs are for GH jobs)...

@trancexpress trancexpress marked this pull request as draft June 11, 2026 13:23
@trancexpress trancexpress force-pushed the gh4009 branch 2 times, most recently from 6fb2161 to b57c207 Compare June 12, 2026 06:50
Tracing for fail in QuickAccessDialogTest hints at a disposed table
prior to updating entries in the table. This change adds tracing
for table dispose to validate this.

See: eclipse-platform#4009
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant