[fix](doc) dev: correct four wrong SQL-function examples (trim_in / murmur_hash3_64_v2 / json_extract_bigint / uncompress)#3864
Open
boluor wants to merge 1 commit into
Conversation
…urmur_hash3_64_v2 / json_extract_bigint / uncompress)
Four dev function-reference examples whose documented output or SQL did not match
actual behavior. Each verified end-to-end on a live master daily build
(doris-0.0.0-2e72603618c); fixes use the real cluster output.
1. EN string-functions/trim-in.md — "Comparison with TRIM function" example
showed trim('ababccaab','ab') = 'ababccaab' (unchanged), which is wrong;
TRIM removes the 'ab' unit from both ends. Corrected to 'cca'. (ZH already
correct.)
SELECT trim_in('ababccaab','ab'), trim('ababccaab','ab');
+----------------------------+-------------------------+
| trim_in('ababccaab', 'ab') | trim('ababccaab', 'ab') |
+----------------------------+-------------------------+
| cc | cca |
+----------------------------+-------------------------+
2. ZH encrypt-digest-functions/murmur-hash3-64-v2.md — first example's result
table (header + values) was copy-pasted from the non-v2 murmur_hash3_64 page.
The query is murmur_hash3_64_v2(...); replaced the table with the v2 output
(mirrors the EN page):
select murmur_hash3_64_v2(null), murmur_hash3_64_v2("hello"), murmur_hash3_64_v2("hello","world");
-> NULL | -3758069500696749310 | -662943091231200135
3. ZH json-functions/json-extract-bigint.md — first example called
json_extract_int (returns NULL on the 122222222222223 value via int overflow)
while the header/expected showed json_extract_bigint. Corrected the call to
json_extract_bigint to match the header, the expected 122222222222223, and EN:
SELECT json_extract_bigint('{"id": 122222222222223, "name": "doris"}', '$.id'); -> 122222222222223
4. ZH string-functions/uncompress.md — third example query was
uncompress(compress(abc)) (abc unquoted -> parser error "Unknown column 'abc'")
while its header/expected showed uncompress('abc') -> NULL. Corrected the query
to uncompress('abc') to match the header, expected, and EN:
select uncompress('abc'); -> NULL
Scope: dev only (docs/ + i18n current/). No ja-source changes.
Co-Authored-By: Claude Opus 4.8 (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
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.
Four dev function-reference examples whose documented output or SQL did not match actual behavior. Each verified end-to-end on a live master daily build (
doris-0.0.0-2e72603618c); fixes use the real cluster output.1. EN
string-functions/trim-in.md— wrongtrim()valueThe "Comparison with TRIM function" example showed
trim('ababccaab','ab')=ababccaab(unchanged), which is wrong —TRIMstrips theabunit from both ends. (ZH page already correct.)2. ZH
encrypt-digest-functions/murmur-hash3-64-v2.md— result table copied from the non-v2 pageThe first example queries
murmur_hash3_64_v2(...)but the result table (header and values) was copy-pasted frommurmur_hash3_64. Replaced with the v2 output (mirrors the EN page):3. ZH
json-functions/json-extract-bigint.md— wrong function in first exampleCalled
json_extract_int(which returnsNULLfor122222222222223via int overflow) while the header/expected showedjson_extract_bigint. Corrected the call to match the header, the expected value, and the EN page:4. ZH
string-functions/uncompress.md— broken third-example queryQuery was
uncompress(compress(abc))(abcunquoted → parser errorUnknown column 'abc') while its header/expected showeduncompress('abc')→NULL. Corrected to match the header, expected, and EN page:Scope: dev only (
docs/+i18n/.../current/). Noja-sourcechanges. (murmur_hash3_64_v2,json_extract_bigintalso exist in released versions — backport scope will be mapped separately.)🤖 Generated with Claude Code