Fix live snapshot: show bulk inserts, fix decimal cast#304
Merged
erikdarlingdata merged 1 commit intodevfrom Feb 26, 2026
Merged
Fix live snapshot: show bulk inserts, fix decimal cast#304erikdarlingdata merged 1 commit intodevfrom
erikdarlingdata merged 1 commit intodevfrom
Conversation
Use sql_handle instead of plan_handle for dm_exec_sql_text so bulk insert operations (which have no plan_handle) appear in live snapshots. Fix System.Single to System.Decimal cast errors for percent_complete and granted_query_memory_gb in Dashboard's active query reader. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.
Summary
COALESCE(der.sql_handle, der.plan_handle)fordm_exec_sql_textso bulk insert operations appear in live snapshots (they have noplan_handle)System.SingletoSystem.Decimalcast error forpercent_completeandgranted_query_memory_gbin DashboardTest plan
🤖 Generated with Claude Code