Repository navigation
Conversation
… summary stats Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Contributor
📦 TestPyPI package publishedpip install --index-strategy unsafe-best-match --index-url https://test.pypi.org/simple/ --extra-index-url https://pypi.org/simple/ buckaroo==0.15.9.dev37492679921or with uv: uv pip install --index-strategy unsafe-best-match --index-url https://test.pypi.org/simple/ --extra-index-url https://pypi.org/simple/ buckaroo==0.15.9.dev37492679921MCP server for Claude Codeclaude mcp add buckaroo-table -- uvx --from "buckaroo[mcp]==0.15.9.dev37492679921" --index-strategy unsafe-best-match --index-url https://test.pypi.org/simple/ --extra-index-url https://pypi.org/simple/ buckaroo-table📖 Docs preview🎨 Storybook preview |
This was referenced Oct 6, 2026
Open
…and narrows the D11 cache-id gap Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
Nothing on main decides, from the size of a dataframe, how much of the stats pipeline to run, and the hard-coded gates that exist apply a 50,000-row sample to every stat, including
length,null_count,minandmax, which are then wrong above 1M cells and 50,000 rows. On large xorq entries stats take 75 to 215 s. The policy PRs (#1019, #1023, #1029, #1031, #1033, #1034, and the client PRs #1032 and #1036) implement a three-tier subset (schema,scalar,full). The wider design, with anapproxtier, a per-stat tier declaration and a footer provider, exists only in design notes outside the repo.Change
Adds
docs/plans/ADR-003-stats-tiers-and-size-policy.md, status Proposed. It records twelve decisions: four ordered tiers; exact scalars that never read the 50,000-row sample;tierandapproxonStatFuncwith a fail-closed default; the filter at the start ofbuild_column_dag; oneStatsLimitstable with a fixed configuration order; a server-side policy with a ceiling no caller can exceed;omitted_keysandapprox_keysindf_meta.stats; the compute control, cost pause and demand columns; sort and search guards; the parquet footer rung; tier in every cache key; runtime-added stats. It also lists the alternatives rejected and the 13 decisions that are still open, each with a recommendation where there is one.No code changes.
Stack
The PRs listed above are retargeted to this branch as their base. Merging this PR retargets them to
main. The branch name has no slash because the Checks workflow'spull_requesttrigger usesbranches: "*", which does not match a base branch containing a slash.Companion ADRs: ADR-001 (#1040, the stat cache) and ADR-002 (how stats reach the client).
🤖 Generated with Claude Code