Skip to content

Commit

Permalink
Fix date math and improve MI compatibility
Browse files Browse the repository at this point in the history
This pull request looks a little funny, because when I re-synced my fork, the available plans query was all beat up for some reason, and I had to paste in the version from the current master branch.

Anyway, the two actual changes in this PR:
 * Closes BrentOzarULTD#3385
 * Closes BrentOzarULTD#3392

Since I can't repro BrentOzarULTD#3385, I'm making the smallest change possible that fixes the issue with mocked-up test data. It will need some community testing to validate.

For BrentOzarULTD#3392 I added a local variable to detect MI, and then set the TargetSessionType to ring_buffer if it's NULL, and we're looking at the system_health extended event. It might be a good idea to have this happen for any session on MI, but I'm going to keep it limited at first.
  • Loading branch information
erikdarlingdata committed Nov 20, 2023
1 parent 7700359 commit 019d4e4
Showing 1 changed file with 81 additions and 56 deletions.
Loading

0 comments on commit 019d4e4

Please sign in to comment.