Repository navigation
Web: the system-health, Blocking, Deadlocks and PostgreSQL event panels say where their data starts (part of #4966) - #5222
Merged
Merged
Conversation
…locks (#4966) Adds get_pg_deadlocks, get_pg_log_events, get_blocking, get_deadlocks and get_deadlock_detail to the data-start note. Event-time rule plus capped rule; get_blocking probes both capture tables and the earlier start wins, as the tool does.
…to HEAD # Conflicts: # Darling/Darling.Tests/WebDataStartNoteConfigAndLogTests.cs # Darling/Darling.Tests/WebDataStartNoteEventReadsTests.cs # Darling/Darling.Tests/WebDataStartNoteLiveTests.cs # Darling/PerformanceMonitor.Darling.Service/WebDataStartNote.cs
…ool arguments by name
erikdarlingdata
marked this pull request as ready for review
October 5, 2026 15:48
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What users see change
The Darling server page's "partial window" note now covers fourteen more event reads, on data answers and on empty answers alike:
get_health_parser_*reads.get_blocking), Deadlocks (get_deadlocks) and Deadlock Graphs (get_deadlock_detail).get_pg_deadlocks) and Log Events (get_pg_log_events).Before this, an empty System Events panel over a window the store never covered showed nothing. The tool's own keys sat under
hints, which the page doesn't read. Now the web writes its own note, in the browser's zone.How the note is decided
ViewerEventDataStart.Of. A first collection can store events from before itself.total_entries,error_count,issue_count,event_countorwait_count) exceedsshown, and their oldest row is the minimumevent_time.truncatedand its rows'occurred_at; the log page througholdest_returned_at.get_blockingis fed by two tables, the blocked process reports and the DMV blocking snapshots. Its probe asks both, and the earlier start wins, as the MCP tool and the desktop viewer do (a new composite source,CompositeSourcesByRead).empty, PostgreSQL'sno_deadlocksandno_events.unavailablewithsource_observed:false.eventsanddeadlocksare row words for their own read only.Test plan
collection_time:get_blockingwith rows only in the DMV snapshots and only in the XE reports, each naming its own table's start.--no-incrementalbuild: 0 warnings;WebDataStartNote*class plusSeparatelyMonitoredListNoteTests: 135 tests, 0 failed;CHANGELOG
SECTION: Fixed
ENTRY: - The web page's System Events, Blocking, Deadlocks and PostgreSQL event panels say where their data starts ([#5222]) - fourteen event reads now show the partial-window note when the store's history starts after the window starts, on empty answers too: the earlier of the coverage start and the earliest event shown, or a capped page's oldest row. Blocking probes both of its sources, as the desktop does.
REF: [#5222]: #5222
Part of #4966