jacktengg commented on PR #4131:
URL: https://github.com/apache/doris-website/pull/4131#issuecomment-5630688188

   <!-- doris-repo-review:v1:begin -->
   ### Local pipeline review — ✅ PASS
   
   ```yaml
   schema: doris-repo-review/v1
   status: PASS
   pr: apache/doris-website#4131
   commit: 77642cad03cfa1b081b64b1ab05687e2328fd900
   base: 62f924204ae749d467acc5e59460a83142a8baa7
   reviewed_at: 2026-09-11T14:57+08:00
   reviewer: jacktengg
   model: gpt-6-astra
   effort: xhigh
   findings: {blocker: 0, major: 0, minor: 3, nit: 0}
   rounds: 1
   converged: true
   ```
   
   **Notes for maintainers**
   
   - Minor: correct `28-bit` to `128-bit` in [the English type 
overview](https://github.com/apache/doris-website/blob/77642cad03cfa1b081b64b1ab05687e2328fd900/docs/sql-manual/basic-element/sql-data-types/data-type-overview.md#L107).
   - Minor: [ARRAY now advertises UUID 
support](https://github.com/apache/doris-website/blob/77642cad03cfa1b081b64b1ab05687e2328fd900/docs/sql-manual/basic-element/sql-data-types/semi-structured/ARRAY.md#L185);
 add development-version UUID to the ARRAYS_OVERLAP and COUNTEQUAL parameter 
inventories in both current locales.
   - Minor: remove the optional-fallback sentence from TO_UUID_OR_NULL and 
TO_UUID_OR_ZERO in both locales; [these 
notes](https://github.com/apache/doris-website/blob/77642cad03cfa1b081b64b1ab05687e2328fd900/docs/sql-manual/sql-functions/scalar-functions/uuid-functions/to-uuid-or-null.md#L21)
 contradict their unary signatures. Keep it for TO_UUID_OR_DEFAULT.
   - Static review covered all 160 changed files, including English/Chinese 
consistency, current-only version scope, navigation, examples and 
implementation paths. No builds or tests were run.
   - Implementation evidence is pinned to [apache/doris#67627 at 
4e23d25](https://github.com/apache/doris/commit/4e23d25eb771d96ff3d46d3186e78cc11f35442f);
 that PR was still open when inspected, so coordinate publication with its 
implementation.
   
   <sub>Reviewed locally with the `doris-repo-review` pipeline. Repository 
policy may accept this receipt for the matching commit; it is not a human 
Apache approval.</sub>
   <!-- doris-repo-review:v1:end -->
   


-- 
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