Skip to content

fix: correct query result memory estimate - #287

Merged
chrispader merged 5 commits into
mainfrom
codex/fix-query-result-memory-estimate
Jul 23, 2026
Merged

fix: correct query result memory estimate#287
chrispader merged 5 commits into
mainfrom
codex/fix-query-result-memory-estimate

Conversation

@chrispader

@chrispader chrispader commented Jul 14, 2026

Copy link
Copy Markdown
Member

Summary

  • correct unordered_map row node memory accounting to add per-node padding instead of multiplying by it

Validation

  • xcodebuild Debug iOS simulator app: BUILD SUCCEEDED
  • nr typecheck
  • nr lint-cpp
  • nr sqlite typecheck
  • git diff --check

Fixes the issue reported in #261 (comment)

@chrispader
chrispader marked this pull request as ready for review July 20, 2026 09:21
@chrispader
chrispader requested a review from mrousavy July 23, 2026 13:53
@chrispader
chrispader merged commit 1945cf5 into main Jul 23, 2026
4 of 5 checks passed
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