Commit db7a0b8
Align sp_HealthParser 2017+ XE time filter to half-open interval
The pre-2017 XML-shred branch used a half-open time filter:
WHERE ca.utc_timestamp >= @start_date
AND ca.utc_timestamp < @end_date
The 2017+ branch used BETWEEN, which is a closed interval — an event
captured at exactly @end_date was included on 2017+ and excluded on
pre-2017. Same inputs, same data, different results depending on
which branch ran.
Rewrote the 2017+ @time_filter to use explicit `>= AND <` so the
boundary semantics match across versions. Half-open is the more
common convention for time windows in this sproc and avoids
double-counting when callers chain windows end-to-end.
Verified sp_HealthParser installs clean and @what_to_check = 'waits'
runs without errors on SQL Server 2022 (a 2017+ build).
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>1 parent 2f3544b commit db7a0b8
1 file changed
Lines changed: 12 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
435 | 435 | | |
436 | 436 | | |
437 | 437 | | |
438 | | - | |
| 438 | + | |
| 439 | + | |
| 440 | + | |
| 441 | + | |
| 442 | + | |
| 443 | + | |
| 444 | + | |
| 445 | + | |
439 | 446 | | |
440 | 447 | | |
441 | 448 | | |
442 | 449 | | |
443 | | - | |
| 450 | + | |
| 451 | + | |
444 | 452 | | |
445 | 453 | | |
446 | | - | |
| 454 | + | |
| 455 | + | |
447 | 456 | | |
448 | 457 | | |
449 | 458 | | |
| |||
0 commit comments