Skip to content

Estimated plan info on Runtime card (#215 E4, E5, E6)#273

Merged
erikdarlingdata merged 1 commit into
devfrom
fix/estimated-plan-info-e4-e6
Apr 24, 2026
Merged

Estimated plan info on Runtime card (#215 E4, E5, E6)#273
erikdarlingdata merged 1 commit into
devfrom
fix/estimated-plan-info-e4-e6

Conversation

@erikdarlingdata
Copy link
Copy Markdown
Owner

Summary

  • E6: MemoryGrantResult exposes DesiredKB + SerialRequiredKB. Estimated plans with a desired grant > 0 now show a "Memory (estimated)" row with the optimizer's desired amount, plus "Serial required" when it differs.
  • E4: compile time already renders on estimated plans when > 0 (existing behavior). Joe's cursor plan has CompileTimeMs=0 on its inner ops, so nothing displays there — that's the actual data.
  • E5: predicted DOP already shows when > 0 (existing). Host-level EstimatedAvailableDOP deliberately not shown — it describes the host, not the plan, and would mislead for explicitly-serial plans.

Part of the v1.8.0 batch.

🤖 Generated with Claude Code

- E6: MemoryGrantResult now exposes DesiredKB (optimizer's pre-execution
  parallel-adjusted desired grant) and SerialRequiredKB. When QueryTime is
  null (estimated plan) but DesiredKB > 0, the Runtime card shows a "Memory
  (estimated)" row with the desired grant, plus a "Serial required" row
  when it differs. Applied to HTML export + web viewer.

- E4: compile time already renders on estimated plans when CompileTimeMs > 0
  (existing behavior). Joe's cursor plan legitimately has compile_time=0 on
  its inner cursor ops, so nothing displays there. No code change needed.

- E5: predicted DOP already shows on estimated plans when DegreeOfParallelism
  > 0 (existing behavior). Plans with no DOP show the Serial reason. Host
  EstimatedAvailableDOP from OptimizerHardwareDependentProperties intentionally
  not surfaced — it describes the host, not the plan, and is misleading for
  plans the optimizer explicitly kept serial.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@erikdarlingdata erikdarlingdata merged commit b7ef358 into dev Apr 24, 2026
2 checks passed
@erikdarlingdata erikdarlingdata deleted the fix/estimated-plan-info-e4-e6 branch April 24, 2026 20:06
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