[fix](doc) SHOW-PARTITION.md en: mark example output illustrative, align fence with zh#3813
Open
boluor wants to merge 1 commit into
Open
[fix](doc) SHOW-PARTITION.md en: mark example output illustrative, align fence with zh#3813boluor wants to merge 1 commit into
boluor wants to merge 1 commit into
Conversation
…ign with zh fence The example uses a hard-coded partition ID `13004` that does not exist on any fresh cluster, so running the snippet returns an empty result set — but the EN page presented its sample 1-row table as the literal expected output (fenced `text`), implying it is reproducible. ZH page already fenced the same result as `sql`, which renders the same in docs but signals "look-like" rather than "literal expected". Update EN to: - Replace "Results:" with prose that calls out that `13004` is illustrative and you must substitute your own partition ID (point at `SHOW PARTITIONS`), and that the function returns an empty set if no partition matches. - Align the fence to ```` ```sql ```` matching ZH. Verified on a single-node Apache Doris 4.1.1 cluster: `SHOW PARTITION 13004` returns 0 rows, no error. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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
1 participant
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.
Summary
Doc page (4.x): `sql-statements/table-and-view/table/SHOW-PARTITION.md` (EN).
The example uses a hard-coded partition ID `13004` and shows a 1-row sample output as if it were the literal expected result. On any fresh cluster (e.g. our Apache Doris 4.1.1 verification rig) that partition does not exist, so the actual output is an empty result set. The doc was internally over-promising. ZH already fenced the same result as ````sql````, which renders the same and signals "look-like" rather than "literal expected".
Two adjustments to EN:
Verification
On a single-node Apache Doris 4.1.1 cluster:
```
mysql> SHOW PARTITION 13004;
Empty set (0.01 sec)
```
A longer-term fix would be to add a setup block that creates a table, calls `SHOW PARTITIONS` to capture a real partition ID, and then uses that ID in `SHOW PARTITION`. That is a much bigger restructure; this PR is the minimum honest fix.
Test plan
🤖 Generated with Claude Code