You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by GitBox <gi...@apache.org> on 2021/08/13 06:37:47 UTC

[GitHub] [incubator-doris] EmmyMiao87 commented on a change in pull request #6430: [Doc]Update hit-the-rollup.md

EmmyMiao87 commented on a change in pull request #6430:
URL: https://github.com/apache/incubator-doris/pull/6430#discussion_r688278713



##########
File path: docs/zh-CN/getting-started/hit-the-rollup.md
##########
@@ -31,7 +31,7 @@ under the License.
 *	索引
 *	聚合数据(仅用于聚合模型,即aggregate key)
 
-但是为了命中 Rollup 需要满足一定的条件,并且可以通过执行计划中 ScanNode 节点的 PreAggregation 的值来判断是否可以命中 Rollup,以及 Rollup 字段来判断命中的是哪一张 Rollup 表。
+但是为了命中 Rollup 需要满足一定的条件,可以通过执行计划中的 Rollup 字段来判断是否命中Rollup以及命中的是哪一张 Rollup 表。

Review comment:
       rollup 字段是为了判断是否命中的。 pre aggregation 字段是用来判断是否可以命中的。

##########
File path: docs/zh-CN/getting-started/hit-the-rollup.md
##########
@@ -117,12 +117,14 @@ Rollup:一般指基于 Base 表创建的 Rollup 表,但在一些场景包括
 +---------------+-------+--------------+------+-------+---------+-------+
 ```
 
-三张表的前缀索引分别为
+这五张表的前缀索引分别为

Review comment:
       英文也一起改一下~




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