Skip to content

Log resource deltas during MarkerTest.testMarkerDeltasMoveFile#2611

Merged
trancexpress merged 1 commit intoeclipse-platform:masterfrom
trancexpress:gh2569_log_deltas
Apr 7, 2026
Merged

Log resource deltas during MarkerTest.testMarkerDeltasMoveFile#2611
trancexpress merged 1 commit intoeclipse-platform:masterfrom
trancexpress:gh2569_log_deltas

Conversation

@trancexpress
Copy link
Copy Markdown
Contributor

@trancexpress trancexpress commented Apr 5, 2026

MarkerTest.testMarkerDeltasMoveFile fails infrequently, due to unexpected deltas received by the test resource change listener. The expected cause is a race condition, likely a parallel job notifying about the resource delta instead of the test main thread.

This change adds logging for resource deltas, including the thread which notifies about the resource delta.
This information hopefully helps to identify the potential race condition.

See: #2569

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 5, 2026

Test Results

    54 files  ±0      54 suites  ±0   36m 59s ⏱️ + 1m 19s
 4 556 tests ±0   4 533 ✅ ±0   23 💤 ±0  0 ❌ ±0 
12 264 runs  ±0  12 105 ✅ ±0  159 💤 ±0  0 ❌ ±0 

Results for commit 2d8a90a. ± Comparison against base commit a43ab52.

♻️ This comment has been updated with latest results.

MarkerTest.testMarkerDeltasMoveFile fails infrequently,
due to unexpected deltas received by the test resource change listener.
The expected cause is a race condition, likely a parallel job
notifying about the resource delta instead of the test main thread.

This change adds logging for resource deltas, including the thread
which notifies about the resource delta. This information hopefully
helps to identify the potential race condition.

See: eclipse-platform#2569
@trancexpress trancexpress merged commit 3f8585f into eclipse-platform:master Apr 7, 2026
18 checks passed
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