Copilot commented on code in PR #3445: URL: https://github.com/apache/doris-website/pull/3445#discussion_r2910628237
########## docs/gettingStarted/must-read-before-poc.md: ########## @@ -41,27 +43,21 @@ This is the simplest syntax — Doris defaults to Duplicate Key, a single partit For a POC, **Duplicate Key works for most scenarios**. Switch only if you have a clear need for upsert or pre-aggregation. For a detailed comparison, see [Data Model Overview](../table-design/data-model/overview). -## 2. Key Columns +### 2. Sort Key Review Comment: The section heading was renamed to “Sort Key”, but this paragraph still refers to “Key columns” throughout. To avoid confusing readers, update the terminology to consistently use “Sort Key” (or “sort key columns”) in this explanation (including “first 36 bytes of …”). ```suggestion **Why it matters:** The sort key determines the **physical sort order** on disk. Doris builds a [prefix index](../table-design/index/prefix-index) on the first 36 bytes of the sort key columns, so queries that filter on these columns run significantly faster. However, when a `VARCHAR` column is encountered, the prefix index stops immediately — no subsequent columns are included. So place fixed-size columns (INT, BIGINT, DATE) before VARCHAR to maximize index coverage. ``` ########## i18n/zh-CN/docusaurus-plugin-content-docs/current/gettingStarted/must-read-before-poc.md: ########## @@ -41,27 +43,21 @@ CREATE TABLE my_table POC 阶段，**Duplicate Key 适用于大多数场景**。只有在明确需要更新或预聚合时才切换。详细对比见[数据模型概述](../table-design/data-model/overview)。 Review Comment: In the ZH doc, most headings are primarily Chinese, but this one is English-first (“Sort Key（排序键）”). For consistency/readability in the localized page, consider switching to a Chinese-first form like “排序键（Sort Key）” (or fully Chinese if that matches the rest of the page style). ```suggestion ## 2. 排序键（Sort Key） ``` -- 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]
