René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"[Order 09][P3] Narrow snapshot projection and review snapshot indexes","articleBody":"## Priority order\n\nOrder: 09 of 09\nPriority: P3 performance\n\n## Problem\n\nSnapshot reads use `SELECT *`, and snapshot indexes should be reviewed after the snapshot-candidate query is fixed.\n\nEvidence:\n\n- `src/NEventStore.Persistence.Sql/SqlDialects/CommonSqlStatements.resx:173`\n- `GetSnapshot` uses `SELECT *`.\n- Snapshot materialization needs `BucketId`, `StreamId`, `StreamRevision`, and `Payload`.\n\n## Impact\n\n- Current blast radius is low because `Snapshots` has only four columns.\n- Future schema additions could make this hot path read unnecessary data.\n- Existing snapshot indexes may not match the corrected query shape from #59.\n\n## Mandatory test-first workflow\n\n- Add tests first that verify snapshot materialization still works with explicit projections.\n- Use existing snapshot acceptance tests as the behavioral contract.\n- Add or document query-plan checks before changing provider indexes.\n- Do not change schema/indexes until the expected query behavior is covered.\n\n## Proposed action\n\n- Select explicit snapshot columns instead of `SELECT *`.\n- After #59, review whether `IX_Snapshots_Stream_Revision` should include `BucketId` first or be replaced by `(BucketId, StreamId, StreamRevision)` depending on provider plans.\n\n## Acceptance criteria\n\n- Snapshot reads return the same objects as before.\n- Provider tests pass for the touched SQL dialects.\n- Any index change is justified by a query-plan comparison or clear provider-specific reasoning.\n\nSource: `docs/Project-Analysis-Issue-Drafts.md`.","author":{"url":"https://github.com/AGiorgetti","@type":"Person","name":"AGiorgetti"},"datePublished":"2026-06-11T09:22:06.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/66/NEventStore.Persistence.SQL/issues/66"}