You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by "dataroaring (via GitHub)" <gi...@apache.org> on 2026/03/12 13:19:32 UTC
[PR] [docs] Fix inverted index link in POC guide [doris-website]
dataroaring opened a new pull request, #3459:
URL: https://github.com/apache/doris-website/pull/3459
## Summary
- fix the broken inverted index link in the English POC guide
- fix the same broken link in the Chinese localized POC guide
--
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: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [docs] Fix inverted index link in POC guide [doris-website]
Posted by "Copilot (via GitHub)" <gi...@apache.org>.
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: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [docs] Fix inverted index link in POC guide [doris-website]
Posted by "dataroaring (via GitHub)" <gi...@apache.org>.
dataroaring merged PR #3459:
URL: https://github.com/apache/doris-website/pull/3459
--
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: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org