sandynz commented on a change in pull request #14062:
URL: https://github.com/apache/shardingsphere/pull/14062#discussion_r767754940



##########
File path: docs/document/content/user-manual/shardingsphere-scaling/usage.cn.md
##########
@@ -134,11 +154,11 @@ 
GENERATED_KEY(COLUMN=order_id,TYPE(NAME=snowflake,PROPERTIES("worker-id"=123)))
 );
 ```
 
-**注意**:当前版本不支持通过修改`TableRule`触发迁移。
+**注意**:当前版本不支持通过修改`TableRule`触发迁移。对于同一个逻辑表,不能混合使用这2种格式。
 
 #### 查询所有迁移任务
 
-详情请参见[RAL#弹性伸缩](/cn/user-manual/shardingsphere-proxy/usage/distsql/syntax/ral/ral/#%E5%BC%B9%E6%80%A7%E4%BC%B8%E7%BC%A9)。
+详情请参见[RAL#弹性伸缩](/cn/user-manual/shardingsphere-proxy/usage/distsql/syntax/ral/#%E5%BC%B9%E6%80%A7%E4%BC%B8%E7%BC%A9)。

Review comment:
       I tested the link, seems the link path was changed, could you replace 
all related links to 
`/cn/user-manual/shardingsphere-proxy/distsql/syntax/ral/#弹性伸缩`?

##########
File path: docs/document/content/user-manual/shardingsphere-scaling/build.cn.md
##########
@@ -18,7 +18,14 @@ mvn clean install -Dmaven.javadoc.skip=true 
-Dcheckstyle.skip=true -Drat.skip=tr
 
 或者通过[下载页面]( https://shardingsphere.apache.org/document/current/cn/downloads/ 
)获取安装包。
 
-2. 解压缩 proxy 发布包,修改配置文件 `conf/server.yaml`,这里主要是开启 `scaling` 和 `mode` 配置:
+> Scaling还是实验性质的功能,建议使用master分支最新版本,点击此处[下载最新版本]( 
https://github.com/apache/shardingsphere#nightly-builds )
+
+2. 解压缩 
proxy发布包,修改配置文件`conf/config-sharding.yaml`,配置`schemaName`,`dataSources`,`rules`里的`shardingAlgorithms`和`keyGenerators`,[config-sharding.yaml示例](
 
https://github.com/apache/shardingsphere/blob/master/examples/docker/shardingsphere-proxy/sharding/conf/config-sharding.yaml
 ):
+
+3. 解压缩 proxy 发布包,修改配置文件 `conf/server.yaml`,这里主要是开启 `scaling` 和 `mode` 配置:
+如果配置的Mode是Cluster,需要提前启动对应的`注册中心`。

Review comment:
       "mode" and "Cluster" could be quoted by '`', and "注册中心" quotes could be 
remove.
   And also other places.

##########
File path: docs/document/content/user-manual/shardingsphere-scaling/usage.cn.md
##########
@@ -110,7 +126,11 @@ ADD RESOURCE ds_2 (
 
 
详情请参见[RDL#数据分片](/cn/user-manual/shardingsphere-proxy/usage/distsql/syntax/rdl/rdl-sharding-rule/)。
 
-`SHARDING TABLE RULE`支持2种类型:`TableRule`和`AutoTableRule`。对于同一个逻辑表,不能混合使用这2种格式。
+`SHARDING TABLE 
RULE`支持2种类型:`TableRule`和`AutoTableRule`。对于同一个逻辑表,不能混合使用这2种格式,以下是两种分片规则的对比:
+| 名称         | AutoTableRule(自动分片)                                    | 
TableRule(自定义分片)                                      |
+| ------------ | ------------------------------------------------------------ 
| ------------------------------------------------------------ |
+| 定义         | 5.x版本之后,引入的自动分片技术交由 ShardingSphere 
自动管理分片,用户只需要指定分片数量和使用的数据源,无需再关心表的具体分布详情可参见:https://shardingsphere.apache.org/document/current/cn/features/sharding/concept/sharding/#自动化分片算法
 | 需要自定义分片配置,可设置物理数据节点,数据分片、表分片等   |

Review comment:
       
https://shardingsphere.apache.org/document/current/cn/features/sharding/concept/sharding/#自动化分片算法
 could be replaced to markdown link format.

##########
File path: docs/document/content/user-manual/shardingsphere-scaling/usage.cn.md
##########
@@ -184,13 +204,25 @@ mysql> show scaling status 660152090995195904;
 | PREPARING                                         | 准备中                      
                                  |
 | RUNNING                                           | 运行中                      
                                  |
 | EXECUTE_INVENTORY_TASK                            | 全量迁移中                    
                                 |
-| EXECUTE_INCREMENTAL_TASK                          | 增量迁移中                    
                                 |
-| ALMOST_FINISHED                                   | 基本完成                     
                                  |
-| FINISHED                                          | 已完成                      
                                   |
+| EXECUTE_INCREMENTAL_TASK                          | 增量迁移中                    
                                 | |

Review comment:
       Seems the end of line includes a duplicated "|".
   
   P.S. You could preview the rendered page in markdown editor, make sure every 
element works.

##########
File path: docs/document/content/user-manual/shardingsphere-scaling/build.cn.md
##########
@@ -67,8 +130,13 @@ sh bin/start.sh
 ## 应用配置项
 
 应用现有配置项如下,相应的配置可在 `conf/server.yaml` 中修改:
-
-| 名称           | 说明                                    | 默认值 |
-| -------------- | -------------------------------------- | ------ |
-| blockQueueSize | 数据传输通道队列大小                      | 10000  |
-| workerThread   | 工作线程池大小,允许同时运行的迁移任务线程数 | 40     |
+| 一级Key | 二级Key                       | 三级Key                                  
                    | 说明                                                        
 | 默认值    |
+| ------- | ----------------------------- | 
------------------------------------------------------------ | 
------------------------------------------------------------ | --------- |
+| Scaling |                               | blockQueueSize                     
                          | 数据传输通道队列大小                                         
| 10000     |
+|         |                               | workerThread                       
                          | 工作线程池大小,允许同时运行的迁移任务线程数                 | 40        |
+|         | clusterAutoSwitchAlgorithm    | type                               
                          | 开启自动检测任务是否完成及切换配置,目前系统提供了IDLE类型实现 | -         |
+|         |                               | 
props:incremental-task-idle-minute-threshold                 | 
增量同步空闲时间最大值,超过该值,进入下一状态               | 30 (分钟) |
+|         | dataConsistencyCheckAlgorithm | type                               
                          | 
配置设置数据校验算法,关闭该配置系统将不进行数据校验。目前系统提供了DEFAULT类型实现,DEFAULT算法目前支持的数据库:MySQL。其他数据库还不能打开这个配置项,相关支持还在开发中可以通过ScalingClusterAutoSwitchAlgorithm接口自定义一个SPI实现,通过ScalingDataConsistencyCheckAlgorithm接口自定义一个SPI实现。详情请参见[开发者手册#弹性伸缩](https://shardingsphere.apache.org/document/current/cn/dev-manual/scaling/)。
 | -         |

Review comment:
       Seems "可以通过ScalingClusterAutoSwitchAlgorithm接口自定义一个SPI实现" is duplicated.
   
   https://shardingsphere.apache.org/document/current/cn/dev-manual/scaling/ 
could be replaced to relative link.




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


Reply via email to