Skip to content

[fix](sql-functions) provide setup data for MEDIAN example (dev)#3903

Merged
morningman merged 1 commit into
apache:masterfrom
boluor:fix/phantom-table-setup-median-dev
Jun 10, 2026
Merged

[fix](sql-functions) provide setup data for MEDIAN example (dev)#3903
morningman merged 1 commit into
apache:masterfrom
boluor:fix/phantom-table-setup-median-dev

Conversation

@boluor

@boluor boluor commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

The dev MEDIAN page queried log_statis without defining it, so the example errored with table does not exist; its second example's SQL and printed output were also mismatched.

Port the version-4.x Example section to dev (EN+ZH): a -- setup block that creates log_statis and loads rows whose per-group medians are exactly the 50 / 30 the doc prints (MEDIAN = percentile 0.5 is deterministic), plus the self-consistent NULL example. version-4.x already has this and is unchanged. (MEDIAN only exists in dev + 4.x.)

Verification — executed end-to-end on the local master daily build (doris-0.0.0-2e72603618c): dev EN P4 F0, dev ZH P4 F0, every touched example reproducing the doc's printed output cell-for-cell.

🤖 Generated with Claude Code

The dev MEDIAN page queried `log_statis` without defining it, so the example
errored with "table does not exist"; its second example's SQL and printed
output were also mismatched.

Port the version-4.x Example section to dev (EN+ZH): a `-- setup` block that
creates `log_statis` and loads rows whose per-group medians are exactly the
50 / 30 the doc prints (MEDIAN = percentile 0.5 is deterministic), plus the
self-consistent NULL example. version-4.x already has this and is unchanged.

Verified end-to-end on the local master daily build — every touched example
reproduces the doc's printed output cell-for-cell (dev EN P4 F0, dev ZH P4 F0).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@morningman morningman merged commit 3f84d1e into apache:master Jun 10, 2026
3 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.

2 participants