HappenLee commented on PR #4091:
URL: https://github.com/apache/doris-website/pull/4091#issuecomment-5475315807
## Review 发现
### [P1] 中文 PREVIOUS_DAY 页面实际写成了 NEXT_DAY
i18n/zh-CN/docusaurus-plugin-content-docs/current/sql-manual/sql-functions/scalar-functions/date-time-functions/
previous-day.md:5
该页面的标题、description、正文和语法仍然是 NEXT_DAY,甚至描述为“指定日期之后”,与文件对应的 PREVIOUS_DAY
完全相反。用
户按文档执行会调用错误函数。本 PR 又在这些错误内容中加入了 TIMESTAMP_NS,应一并修正为 PREVIOUS_DAY 和“指定日期之前”。
### [P2] NANOSECOND 漏掉实际支持的重载
docs/sql-manual/sql-functions/scalar-functions/date-time-functions/nanosecond.md:11
文档将参数限定为 TIMESTAMP_NS,但 Doris PR #66761 的函数签名同时支持:
- NANOSECOND(DATE)
- NANOSECOND(DATETIME(p))
- NANOSECOND(TIMESTAMP_NS)
对应实现也有专门的 DATE/DATETIME 回归测试。按照 SQL 函数文档规范,Syntax、Parameters 和 Example
都应该覆盖这些重载。
### [P2] 英文 DATEDIFF 形成了错误的非对称参数说明
docs/sql-manual/sql-functions/scalar-functions/date-time-functions/datediff.md:25
PR 只给 <expr2> 增加了 TIMESTAMP_NS,<expr1> 仍写成只支持 DATE/DATETIME。但实现支持
TIMESTAMP_NS 出现在任意一侧,中文
页面也同时更新了两个参数。
### [P2] 英文 SECONDS_DIFF 的参数表未同步更新
docs/sql-manual/sql-functions/scalar-functions/date-time-functions/seconds-diff.md:23
Description 已声明支持 TIMESTAMP_NS,但两个参数仍然只列出 DATE/DATETIME,和同页描述、中文文档及代码签名不一致。
类似的不完整批量修改还出现在 DATE_SUB、HOUR、部分 *_CEIL/*_FLOOR 页面:概述、参数和返回值没有同步表达支持范围。建议统
一扫描一次所有 190 个被修改的日期函数页面。
### [P2] 中文 MINUTES_DIFF 参数表缺少 <date_or_time_expr2>
i18n/zh-CN/docusaurus-plugin-content-docs/current/sql-manual/sql-functions/scalar-functions/date-time-functions/
minutes-diff.md:23
两行都写成了 <date_or_time_expr1>,第二行应该是 <date_or_time_expr2>。本 PR 修改了这两行但保留了原错误。
--
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]