Copilot commented on code in PR #3459: URL: https://github.com/apache/doris-website/pull/3459#discussion_r2924608473
########## i18n/zh-CN/docusaurus-plugin-content-docs/current/gettingStarted/before-you-start-the-poc.md: ########## @@ -27,7 +27,7 @@ ### Sort Key（排序键） -将最常用于过滤的列放在最前面，定长类型（INT、BIGINT、DATE）放在 VARCHAR 之前。Doris 在排序键的前 36 字节上构建[前缀索引](../table-design/index/prefix-index)，但遇到 VARCHAR 会立即截断。其他需要快速过滤的列可添加[倒排索引](../table-design/index/inverted-index)。 +将最常用于过滤的列放在最前面，定长类型（INT、BIGINT、DATE）放在 VARCHAR 之前。Doris 在排序键的前 36 字节上构建[前缀索引](../table-design/index/prefix-index)，但遇到 VARCHAR 会立即截断。其他需要快速过滤的列可添加[倒排索引](../table-design/index/inverted-index/overview)。 Review Comment: Same as the English doc: this change also removes the leading `|` (and indentation) from the paragraph, which likely changes formatting in the rendered page. If the goal is only to fix the broken inverted index link, consider restoring the original ` |` prefix and only updating the URL; otherwise, document the formatting change in the PR description. ########## docs/gettingStarted/before-you-start-the-poc.md: ########## @@ -27,7 +27,7 @@ Creating a table in Doris involves four decisions that affect load and query per ### Sort Key -Put the column you filter on most frequently first, with fixed-size types (INT, BIGINT, DATE) before VARCHAR. Doris builds a [prefix index](../table-design/index/prefix-index) on the first 36 bytes of key columns but stops at the first VARCHAR. Add [inverted indexes](../table-design/index/inverted-index) for other columns that need fast filtering. +Put the column you filter on most frequently first, with fixed-size types (INT, BIGINT, DATE) before VARCHAR. Doris builds a [prefix index](../table-design/index/prefix-index) on the first 36 bytes of key columns but stops at the first VARCHAR. Add [inverted indexes](../table-design/index/inverted-index/overview) for other columns that need fast filtering. Review Comment: This line change removes the leading `|` (and indentation) that existed previously, which can change the rendered output beyond the stated goal of “fix the broken inverted index link”. If the intent is strictly to update the link target, keep the original leading formatting (` |`) and only change the URL; otherwise, please update the PR description to mention the formatting change and confirm it’s intentional. -- 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]
