[fix](docs) BITMAP_AGG / MAP_AGG example setup (dev + 3.x + 2.1)#3895
Merged
morningman merged 2 commits intoJun 3, 2026
Merged
Conversation
…x/2.1 Both pages' examples query tables the page never creates (test_bitmap_agg, nation), so readers hit "table does not exist". Add a visible CREATE TABLE + INSERT before the first example on each page (EN + ZH, version-3.x and version-2.1), reconstructed from each page's own printed output. Verified end-to-end on live 3.1.4 and 2.1.11 clusters: every example reproduces the documented output exactly (P5 F0 per page), including the BITMAP_AGG VARCHAR->BIGINT cast cases and the MAP_AGG grouped map output. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
The dev BITMAP_AGG page declares `k5 BIGINT` in its setup table but inserts 18446744073709551615 and 18446744073709551616, which exceed the BIGINT range. The whole INSERT is rejected, leaving the table empty, so every example on the page fails. Change `k5` to VARCHAR(32) (and quote its values) so the large values are stored as written; BITMAP_AGG converts them and applies its own range rule (values < 0 or > 18446744073709551615 are dropped), reproducing the documented output. Verified end-to-end on a live master build: every example passes (EN + ZH). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
boluor
changed the title
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
2 participants
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.
Adds the missing CREATE TABLE + INSERT setup so the BITMAP_AGG and MAP_AGG examples are runnable end-to-end.
BIGINT→VARCHAR(32): the INSERT carries values18446744073709551615/616that overflow BIGINT and would reject the whole row, leaving the table empty; BITMAP_AGG converts the string per its domain rule, <0 or >uint64max dropped).nation(25 rows).Verified end-to-end on fresh single-BE clusters (4.1.1 / 3.1.4 / 2.1.11 and local master daily build): every touched example reproduces the doc's printed output cell-for-cell.
This supersedes #3897 (the dev k5 fix is folded in here so BITMAP_AGG lives in one PR).
🤖 Generated with Claude Code