Skip to content

Show nonclustered index count on modification operators#168

Merged
erikdarlingdata merged 1 commit into
devfrom
feature/nc-index-count
Apr 6, 2026
Merged

Show nonclustered index count on modification operators#168
erikdarlingdata merged 1 commit into
devfrom
feature/nc-index-count

Conversation

@erikdarlingdata
Copy link
Copy Markdown
Owner

Summary

  • Displays a +N NC badge on INSERT/UPDATE/DELETE operators when they maintain nonclustered indexes
  • Tooltip shows index names (e.g., "a0, a1, a2, a3, a4")
  • Properties panel lists each index individually under "NC Indexes Maintained"
  • Only counts on modification operators (Update/SimpleUpdate/CreateIndex elements), not read operators
  • Clustered index or heap is implicit and not counted

Closes #167.

Test plan

  • UPDATE plan with 5 NC indexes → badge shows "+5 NC", tooltip and properties list names
  • INSERT plan with 5 NC indexes → same
  • DELETE plan with 5 NC indexes → same (uses SimpleUpdate element)
  • Read-only plan (key lookup) → no badge, zero count on all nodes
  • 4 unit tests pass
  • Full test suite (64 tests) passes

🤖 Generated with Claude Code

Display a badge on INSERT/UPDATE/DELETE operators showing how many
nonclustered indexes are maintained. The clustered index or heap is
implicit; the NC count surfaces the hidden maintenance overhead.

- Parse Object elements with IndexKind="NonClustered" inside
  Update/SimpleUpdate/CreateIndex operator elements
- Show "+N NC" badge on the operator node
- List index names in tooltip and properties panel
- Only counts on modification operators, not read operators
- 4 unit tests with INSERT, UPDATE, DELETE, and read-only plans

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@erikdarlingdata erikdarlingdata merged commit 4b5dc54 into dev Apr 6, 2026
1 check passed
@erikdarlingdata erikdarlingdata deleted the feature/nc-index-count branch April 9, 2026 00:37
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