wangtianyi2004 commented on code in PR #1407: URL: https://github.com/apache/doris-website/pull/1407#discussion_r1886136159
########## i18n/zh-CN/docusaurus-plugin-content-docs/current/admin-manual/workload-mangement/compute-group.md: ########## Review Comment: 1. 大写 ADD BE  2. 缺少分号  3. 在文档最后添加一句详细操作参考存算分离相关操作(给一个链接)。 ########## i18n/zh-CN/docusaurus-plugin-content-docs/current/admin-manual/workload-mangement/workload-group.md: ########## Review Comment: 需要添加版本说明  #版本升级说明 自 Doris 2.0 版本开始提供 Workload 资源管控功能。在 Doris 2.0 版本中,Workload 资源管控不依赖于 CGroup,而 Doris 2.1 版本中需要依赖 CGroup。 - 从 Doris 1.2 升级到 2.0:建议集群升级完成后,再开启 Workload 资源管控功能。只升级部分 follower FE 节点,可能会因为未升级的 FE 节点没有 Workload Group 的元数据信息,导致已升级的 follower FE 节点查询失败。 - 从 Doris 2.0 升级到 2.1:由于 2.1 版本的 Workload 资源管控功能依赖于 CGroup,需要先配置 CGroup 环境,在升级到 Doris 2.1 版本。 ########## i18n/zh-CN/docusaurus-plugin-content-docs/current/admin-manual/workload-mangement/kill-query.md: ########## Review Comment: LGTM ########## i18n/zh-CN/docusaurus-plugin-content-docs/current/admin-manual/workload-mangement/sql-blocking.md: ########## Review Comment: Typo    ########## i18n/zh-CN/docusaurus-plugin-content-docs/current/admin-manual/workload-mangement/sql-blocking.md: ########## Review Comment: 使用方法标题下,应该有四个标题 - 阻止扫描超过指定行数的数据 - 阻止扫描超过指定分数数量的数据 - 阻止扫描超过指定分桶数量的数据 - 阻止特定模式的查询 存在以下问题: 1. 当前文档标题级别没有对齐 2. 在使用场景前,增加一句描述,可以在这四个场景使用 ########## i18n/zh-CN/docusaurus-plugin-content-docs/current/admin-manual/workload-mangement/sql-blocking.md: ########## Review Comment: 使用方法标题下,应该有四个标题 - 阻止扫描超过指定行数的数据 - 阻止扫描超过指定分数数量的数据 - 阻止扫描超过指定分桶数量的数据 - 阻止特定模式的查询 存在以下问题: 1. 当前文档标题级别没有对齐 2. 在使用场景前,增加一句描述,可以在这四个场景使用 ########## i18n/zh-CN/docusaurus-plugin-content-docs/current/admin-manual/workload-mangement/workload-management-summary.md: ########## Review Comment: LGTM -- 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]
