Skip to content

Use space-saving counters for column TopN#364

Closed
KKould wants to merge 1 commit into
result-tuple-visitor-tpccfrom
codex/topn-space-saving
Closed

Use space-saving counters for column TopN#364
KKould wants to merge 1 commit into
result-tuple-visitor-tpccfrom
codex/topn-space-saving

Conversation

@KKould

@KKould KKould commented Jun 22, 2026

Copy link
Copy Markdown
Member

Summary

  • update ColumnTopN insertion to use Space-Saving replacement when capacity is full
  • preserve over-estimation error from replaced minimum counters
  • update TopN tests for replacement and merge error behavior

Tests

  • cargo fmt --check
  • cargo check -q
  • cargo test -q --features decimal top_n

@KKould

KKould commented Jun 22, 2026

Copy link
Copy Markdown
Member Author

Superseded by the combined PR #366.

@KKould KKould closed this Jun 22, 2026
@KKould KKould deleted the codex/topn-space-saving branch June 22, 2026 04:45
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