Skip to content

[fix](sql-functions) reconstruct example1 setup for EXPLODE_SPLIT (version-3.x / version-2.1)#3890

Merged
morningman merged 1 commit into
apache:masterfrom
boluor:fix/lineB-explode-split-setup-3x-21
Jun 3, 2026
Merged

[fix](sql-functions) reconstruct example1 setup for EXPLODE_SPLIT (version-3.x / version-2.1)#3890
morningman merged 1 commit into
apache:masterfrom
boluor:fix/lineB-explode-split-setup-3x-21

Conversation

@boluor
Copy link
Copy Markdown
Contributor

@boluor boluor commented Jun 2, 2026

The EXPLODE_SPLIT pages early examples query example1, which the version-3.x / version-2.1 docs never define (the page only defines example2, used by the later examples). So those examples cannot be run or reproduced (Table [...] does not exist).

This PR adds the missing example1 CREATE TABLE + INSERT, reconstructed from the pages own select * from example1 output, as a visible inline block before the first example that uses it.

Verified on version-3.x (3.1.4) and version-2.1 (2.1.11): all but one example resolve and match the documented output. EN + ZH (4 files). No ja-source changes.

One example has a pre-existing output bug — its rows are not sorted despite ORDER BY e1 (the engine correctly returns 1,2,3 but the doc shows 2,3,1). That is not addressed here; it is tracked in #3889.

…rsion-3.x / version-2.1)

The EXPLODE_SPLIT page's early examples query `example1`, which the version-3.x /
version-2.1 docs never define (the page only defines `example2`, used later). So
those examples cannot be run or reproduced ("table does not exist").

Add the missing `example1` CREATE TABLE + INSERT, reconstructed from the page's
own `select * from example1` output, as a visible inline block before the first
example that uses it.

Verified on version-3.x (3.1.4) and version-2.1 (2.1.11): all but one example
resolve and match the documented output. The remaining example has a pre-existing
output bug (its rows are not sorted despite `ORDER BY e1`), tracked in apache#3889 and
left for a separate doc correction.

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