boluor opened a new issue, #3896:
URL: https://github.com/apache/doris-website/issues/3896

   ### Pages
   
   `COLLECT_LIST` and `COLLECT_SET` function references — `version-3.x` and 
`version-2.1`, EN + ZH:
   - 
`versioned_docs/version-3.x/sql-manual/sql-functions/aggregate-functions/collect-list.md`
   - `versioned_docs/version-3.x/.../collect-set.md` (+ version-2.1 and the 
`i18n/zh-CN` copies)
   
   ### Problems
   
   **1. Undefined tables.** The examples query `collect_list_test` / 
`collect_set_test`, but neither page ever creates them, so copy-pasting fails 
with `table does not exist`.
   
   **2. Outdated ARRAY display format.** Even after defining the tables, the 
printed outputs use a legacy array rendering that current Doris (verified on 
3.1.4 and 2.1.11) no longer produces:
   
   - **String/date elements are shown unquoted**, e.g. the docs print 
`[hello]`, `[world]`, `[2023-01-01,2023-01-02]`, but Doris renders string/date 
arrays **quoted**: `["hello"]`, `["world"]`, `["2023-01-01", "2023-01-02"]`.
   - **No space after the comma**, e.g. docs `[1,2,2,3,3,4,4]` vs Doris `[1, 2, 
2, 3, 3, 4, 4]`.
   
   (Note this is internally inconsistent already — e.g. the sibling `ARRAY_AGG` 
page prints quoted string arrays.)
   
   **3. `COLLECT_SET` extras.** The set-ordering example prints `[4,3,2,1]`, 
but `COLLECT_SET` returns an unordered set — element order is not 
deterministic, so the literal output isn't reproducible. The same example's 
output table also has a malformed bottom border (`+----...----+` merged across 
both columns).
   
   ### Suggested fix
   
   Add `CREATE TABLE` + `INSERT` setup for both pages **and** refresh the 
expected outputs to the current array rendering (quoted string elements, space 
after comma). For the `COLLECT_SET` ordering example, either sort for 
determinism (e.g. `array_sort(collect_set(...))`) or relax the expected output, 
and fix the table border.
   
   Left out of the phantom-table-setup PR series because it needs the 
documented outputs rewritten to match current rendering (an editorial change on 
these older release lines), not just a missing-table addition.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to