René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"[Order 02][P1] Fix async paged-query infinite loop when PageSize is zero","articleBody":"## Priority order\n\nOrder: 02 of 09\nPriority: P1 correctness\n\n## Problem\n\nAsync paged reads can loop forever when the configured page size is `0` and the result set is empty.\n\nEvidence:\n\n- `src/NEventStore.Persistence.Sql/SqlDialects/CommonDbStatement.cs:331`\n- Continuation condition is `while (Dialect.CanPage \u0026\u0026 recordsRead == PageSize)`.\n- `SqlPersistenceEngine` allows `PageSize == 0`, where zero represents infinite page size.\n\n## Impact\n\n- Empty async reads can hang for pageable dialects.\n- Affected APIs include checkpoint reads, bucket-scoped reads, and stream reads that call `ExecutePagedQueryAsync`.\n\n## Mandatory test-first workflow\n\n- Add tests first that reproduce or verify the issue before changing production code.\n- Include an async empty-result test with configured page size `0`.\n- Cover at least `GetFromAsync(0, observer, token)` and one bucket-scoped async read.\n- The test must fail or time out before the fix and complete after the fix.\n\n## Proposed action\n\n- Base continuation on the computed local `pageSize`.\n- Continue only when `pageSize \u003e 0 \u0026\u0026 recordsRead == pageSize`.\n\n## Acceptance criteria\n\n- Empty async reads complete when `PageSize == 0`.\n- Multi-page async reads still return ordered, complete results when `PageSize \u003e 0`.\n- Sync behavior is unchanged.\n\nSource: `docs/Project-Analysis-Issue-Drafts.md`.","author":{"url":"https://github.com/AGiorgetti","@type":"Person","name":"AGiorgetti"},"datePublished":"2026-06-11T09:20:30.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":1},"url":"https://github.com/60/NEventStore.Persistence.SQL/issues/60"}