You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@shardingsphere.apache.org by GitBox <gi...@apache.org> on 2020/10/11 04:05:21 UTC

[GitHub] [shardingsphere] strongduanmu opened a new pull request #7740: support create, alter, drop, select view statement

strongduanmu opened a new pull request #7740:
URL: https://github.com/apache/shardingsphere/pull/7740


   Fixes #7138.
   
   Changes proposed in this pull request:
   - support create, alter, drop statement
   - add create, alter view validator
   - add view_type in SchemaMetaDataLoader to support select view
   


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [shardingsphere] tristaZero commented on a change in pull request #7740: support create, alter, drop, select view statement

Posted by GitBox <gi...@apache.org>.
tristaZero commented on a change in pull request #7740:
URL: https://github.com/apache/shardingsphere/pull/7740#discussion_r502901062



##########
File path: shardingsphere-features/shardingsphere-sharding/shardingsphere-sharding-route/src/main/java/org/apache/shardingsphere/sharding/route/engine/validator/ShardingStatementValidatorFactory.java
##########
@@ -66,6 +73,25 @@
         if (sqlStatement instanceof CreateProcedureStatement) {
             return Optional.of(new ShardingCreateProcedureStatementValidator());
         }
+        if (sqlStatement instanceof CreateViewStatement) {
+            return Optional.of(new ShardingCreateViewStatementValidator());
+        }
+        if (sqlStatement instanceof AlterViewStatement) {

Review comment:
       If `Create view` SQL can work well for `single table` currently, do you think `ShardingUnconfiguredTablesRoutingEngine` will be a better match?




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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [shardingsphere] codecov-io edited a comment on pull request #7740: support create, alter, drop, select view statement

Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on pull request #7740:
URL: https://github.com/apache/shardingsphere/pull/7740#issuecomment-706653688


   # [Codecov](https://codecov.io/gh/apache/shardingsphere/pull/7740?src=pr&el=h1) Report
   > Merging [#7740](https://codecov.io/gh/apache/shardingsphere/pull/7740?src=pr&el=desc) into [master](https://codecov.io/gh/apache/shardingsphere/commit/e3bb018d4366da9d5f1ba51197ae3403a2596c99?el=desc) will **increase** coverage by `42.36%`.
   > The diff coverage is `69.14%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/shardingsphere/pull/7740/graphs/tree.svg?width=650&height=150&src=pr&token=ZvlXpWa7so)](https://codecov.io/gh/apache/shardingsphere/pull/7740?src=pr&el=tree)
   
   ```diff
   @@              Coverage Diff              @@
   ##             master    #7740       +/-   ##
   =============================================
   + Coverage     32.82%   75.19%   +42.36%     
   + Complexity     2381      521     -1860     
   =============================================
     Files          1417     1409        -8     
     Lines        104857    22471    -82386     
     Branches       9526     4042     -5484     
   =============================================
   - Hits          34415    16896    -17519     
   + Misses        67755     4485    -63270     
   + Partials       2687     1090     -1597     
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/shardingsphere/pull/7740?src=pr&el=tree) | Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | [.../route/engine/type/ShardingRouteEngineFactory.java](https://codecov.io/gh/apache/shardingsphere/pull/7740/diff?src=pr&el=tree#diff-c2hhcmRpbmdzcGhlcmUtZmVhdHVyZXMvc2hhcmRpbmdzcGhlcmUtc2hhcmRpbmcvc2hhcmRpbmdzcGhlcmUtc2hhcmRpbmctcm91dGUvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL3NoYXJkaW5nc3BoZXJlL3NoYXJkaW5nL3JvdXRlL2VuZ2luZS90eXBlL1NoYXJkaW5nUm91dGVFbmdpbmVGYWN0b3J5LmphdmE=) | `77.27% <0.00%> (-1.80%)` | `0.00 <0.00> (ø)` | |
   | [...sql/parser/mysql/visitor/impl/MySQLDDLVisitor.java](https://codecov.io/gh/apache/shardingsphere/pull/7740/diff?src=pr&el=tree#diff-c2hhcmRpbmdzcGhlcmUtc3FsLXBhcnNlci9zaGFyZGluZ3NwaGVyZS1zcWwtcGFyc2VyLWRpYWxlY3Qvc2hhcmRpbmdzcGhlcmUtc3FsLXBhcnNlci1teXNxbC9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvc2hhcmRpbmdzcGhlcmUvc3FsL3BhcnNlci9teXNxbC92aXNpdG9yL2ltcGwvTXlTUUxERExWaXNpdG9yLmphdmE=) | `47.82% <0.00%> (-0.89%)` | `0.00 <0.00> (ø)` | |
   | [.../postgresql/visitor/impl/PostgreSQLDDLVisitor.java](https://codecov.io/gh/apache/shardingsphere/pull/7740/diff?src=pr&el=tree#diff-c2hhcmRpbmdzcGhlcmUtc3FsLXBhcnNlci9zaGFyZGluZ3NwaGVyZS1zcWwtcGFyc2VyLWRpYWxlY3Qvc2hhcmRpbmdzcGhlcmUtc3FsLXBhcnNlci1wb3N0Z3Jlc3FsL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9zaGFyZGluZ3NwaGVyZS9zcWwvcGFyc2VyL3Bvc3RncmVzcWwvdmlzaXRvci9pbXBsL1Bvc3RncmVTUUxERExWaXNpdG9yLmphdmE=) | `93.70% <0.00%> (-0.75%)` | `0.00 <0.00> (ø)` | |
   | [...t/postgresql/ddl/PostgreSQLAlterViewStatement.java](https://codecov.io/gh/apache/shardingsphere/pull/7740/diff?src=pr&el=tree#diff-c2hhcmRpbmdzcGhlcmUtc3FsLXBhcnNlci9zaGFyZGluZ3NwaGVyZS1zcWwtcGFyc2VyLXN0YXRlbWVudC9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvc2hhcmRpbmdzcGhlcmUvc3FsL3BhcnNlci9zcWwvZGlhbGVjdC9zdGF0ZW1lbnQvcG9zdGdyZXNxbC9kZGwvUG9zdGdyZVNRTEFsdGVyVmlld1N0YXRlbWVudC5qYXZh) | `0.00% <0.00%> (ø)` | `0.00 <0.00> (?)` | |
   | [...e/validator/ShardingStatementValidatorFactory.java](https://codecov.io/gh/apache/shardingsphere/pull/7740/diff?src=pr&el=tree#diff-c2hhcmRpbmdzcGhlcmUtZmVhdHVyZXMvc2hhcmRpbmdzcGhlcmUtc2hhcmRpbmcvc2hhcmRpbmdzcGhlcmUtc2hhcmRpbmctcm91dGUvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL3NoYXJkaW5nc3BoZXJlL3NoYXJkaW5nL3JvdXRlL2VuZ2luZS92YWxpZGF0b3IvU2hhcmRpbmdTdGF0ZW1lbnRWYWxpZGF0b3JGYWN0b3J5LmphdmE=) | `17.39% <18.75%> (-5.69%)` | `0.00 <0.00> (ø)` | |
   | [...dialect/handler/ddl/AlterViewStatementHandler.java](https://codecov.io/gh/apache/shardingsphere/pull/7740/diff?src=pr&el=tree#diff-c2hhcmRpbmdzcGhlcmUtc3FsLXBhcnNlci9zaGFyZGluZ3NwaGVyZS1zcWwtcGFyc2VyLXN0YXRlbWVudC9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvc2hhcmRpbmdzcGhlcmUvc3FsL3BhcnNlci9zcWwvZGlhbGVjdC9oYW5kbGVyL2RkbC9BbHRlclZpZXdTdGF0ZW1lbnRIYW5kbGVyLmphdmE=) | `33.33% <33.33%> (ø)` | `0.00 <0.00> (?)` | |
   | [...ialect/handler/ddl/CreateViewStatementHandler.java](https://codecov.io/gh/apache/shardingsphere/pull/7740/diff?src=pr&el=tree#diff-c2hhcmRpbmdzcGhlcmUtc3FsLXBhcnNlci9zaGFyZGluZ3NwaGVyZS1zcWwtcGFyc2VyLXN0YXRlbWVudC9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvc2hhcmRpbmdzcGhlcmUvc3FsL3BhcnNlci9zcWwvZGlhbGVjdC9oYW5kbGVyL2RkbC9DcmVhdGVWaWV3U3RhdGVtZW50SGFuZGxlci5qYXZh) | `33.33% <33.33%> (ø)` | `0.00 <0.00> (?)` | |
   | [...ator/impl/ShardingAlterViewStatementValidator.java](https://codecov.io/gh/apache/shardingsphere/pull/7740/diff?src=pr&el=tree#diff-c2hhcmRpbmdzcGhlcmUtZmVhdHVyZXMvc2hhcmRpbmdzcGhlcmUtc2hhcmRpbmcvc2hhcmRpbmdzcGhlcmUtc2hhcmRpbmctcm91dGUvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL3NoYXJkaW5nc3BoZXJlL3NoYXJkaW5nL3JvdXRlL2VuZ2luZS92YWxpZGF0b3IvaW1wbC9TaGFyZGluZ0FsdGVyVmlld1N0YXRlbWVudFZhbGlkYXRvci5qYXZh) | `90.90% <90.90%> (ø)` | `0.00 <0.00> (?)` | |
   | [...tor/impl/ShardingCreateViewStatementValidator.java](https://codecov.io/gh/apache/shardingsphere/pull/7740/diff?src=pr&el=tree#diff-c2hhcmRpbmdzcGhlcmUtZmVhdHVyZXMvc2hhcmRpbmdzcGhlcmUtc2hhcmRpbmcvc2hhcmRpbmdzcGhlcmUtc2hhcmRpbmctcm91dGUvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL3NoYXJkaW5nc3BoZXJlL3NoYXJkaW5nL3JvdXRlL2VuZ2luZS92YWxpZGF0b3IvaW1wbC9TaGFyZGluZ0NyZWF0ZVZpZXdTdGF0ZW1lbnRWYWxpZGF0b3IuamF2YQ==) | `90.90% <90.90%> (ø)` | `0.00 <0.00> (?)` | |
   | [...impl/ShardingCreateFunctionStatementValidator.java](https://codecov.io/gh/apache/shardingsphere/pull/7740/diff?src=pr&el=tree#diff-c2hhcmRpbmdzcGhlcmUtZmVhdHVyZXMvc2hhcmRpbmdzcGhlcmUtc2hhcmRpbmcvc2hhcmRpbmdzcGhlcmUtc2hhcmRpbmctcm91dGUvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL3NoYXJkaW5nc3BoZXJlL3NoYXJkaW5nL3JvdXRlL2VuZ2luZS92YWxpZGF0b3IvaW1wbC9TaGFyZGluZ0NyZWF0ZUZ1bmN0aW9uU3RhdGVtZW50VmFsaWRhdG9yLmphdmE=) | `90.90% <100.00%> (-4.75%)` | `0.00 <0.00> (ø)` | |
   | ... and [73 more](https://codecov.io/gh/apache/shardingsphere/pull/7740/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/shardingsphere/pull/7740?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/shardingsphere/pull/7740?src=pr&el=footer). Last update [e3bb018...d40f65b](https://codecov.io/gh/apache/shardingsphere/pull/7740?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [shardingsphere] strongduanmu commented on a change in pull request #7740: support create, alter, drop, select view statement

Posted by GitBox <gi...@apache.org>.
strongduanmu commented on a change in pull request #7740:
URL: https://github.com/apache/shardingsphere/pull/7740#discussion_r503646561



##########
File path: shardingsphere-infra/shardingsphere-infra-common/src/main/java/org/apache/shardingsphere/infra/metadata/model/schema/model/schema/SchemaMetaDataLoader.java
##########
@@ -42,6 +42,8 @@
     
     private static final String TABLE_TYPE = "TABLE";
     
+    private static final String VIEW_TYPE = "VIEW";

Review comment:
       > Why do we need `VIEW_TYPE`?
   
   @tristaZero In order to support select view statement.




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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [shardingsphere] strongduanmu commented on a change in pull request #7740: support create, alter, drop, select view statement

Posted by GitBox <gi...@apache.org>.
strongduanmu commented on a change in pull request #7740:
URL: https://github.com/apache/shardingsphere/pull/7740#discussion_r502917588



##########
File path: shardingsphere-features/shardingsphere-sharding/shardingsphere-sharding-route/src/main/java/org/apache/shardingsphere/sharding/route/engine/validator/ShardingStatementValidatorFactory.java
##########
@@ -66,6 +73,25 @@
         if (sqlStatement instanceof CreateProcedureStatement) {
             return Optional.of(new ShardingCreateProcedureStatementValidator());
         }
+        if (sqlStatement instanceof CreateViewStatement) {
+            return Optional.of(new ShardingCreateViewStatementValidator());
+        }
+        if (sqlStatement instanceof AlterViewStatement) {

Review comment:
       > If `Create view` SQL can work well for `single table` currently, do you think `ShardingUnconfiguredTablesRoutingEngine` will be a better match?
   
   @tristaZero `ShardingUnconfiguredTablesRoutingEngine` can meet the needs of creating views on a single data source. I think it is a better choice. 👍
   




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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [shardingsphere] tristaZero merged pull request #7740: support create, alter, drop, select view statement

Posted by GitBox <gi...@apache.org>.
tristaZero merged pull request #7740:
URL: https://github.com/apache/shardingsphere/pull/7740


   


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [shardingsphere] tristaZero commented on a change in pull request #7740: support create, alter, drop, select view statement

Posted by GitBox <gi...@apache.org>.
tristaZero commented on a change in pull request #7740:
URL: https://github.com/apache/shardingsphere/pull/7740#discussion_r503643226



##########
File path: shardingsphere-features/shardingsphere-sharding/shardingsphere-sharding-route/src/main/java/org/apache/shardingsphere/sharding/route/engine/validator/dml/ShardingDMLStatementValidator.java
##########
@@ -0,0 +1,41 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.shardingsphere.sharding.route.engine.validator.dml;
+
+import org.apache.shardingsphere.infra.binder.statement.SQLStatementContext;
+import org.apache.shardingsphere.infra.binder.type.TableAvailable;
+import org.apache.shardingsphere.infra.exception.ShardingSphereException;
+import org.apache.shardingsphere.sharding.route.engine.validator.ShardingStatementValidator;
+import org.apache.shardingsphere.sql.parser.sql.common.statement.SQLStatement;
+
+/**
+ * Sharding dml statement validator.
+ */
+public abstract class ShardingDMLStatementValidator<T extends SQLStatement> implements ShardingStatementValidator<T> {

Review comment:
       Oh, you did it!

##########
File path: shardingsphere-infra/shardingsphere-infra-common/src/main/java/org/apache/shardingsphere/infra/metadata/model/schema/model/schema/SchemaMetaDataLoader.java
##########
@@ -42,6 +42,8 @@
     
     private static final String TABLE_TYPE = "TABLE";
     
+    private static final String VIEW_TYPE = "VIEW";

Review comment:
       Why do we need `VIEW_TYPE`?




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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [shardingsphere] strongduanmu commented on a change in pull request #7740: support create, alter, drop, select view statement

Posted by GitBox <gi...@apache.org>.
strongduanmu commented on a change in pull request #7740:
URL: https://github.com/apache/shardingsphere/pull/7740#discussion_r502916569



##########
File path: shardingsphere-features/shardingsphere-sharding/shardingsphere-sharding-route/src/main/java/org/apache/shardingsphere/sharding/route/engine/validator/util/StatementValidatorUtil.java
##########
@@ -0,0 +1,83 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.shardingsphere.sharding.route.engine.validator.util;
+
+import lombok.AccessLevel;
+import lombok.NoArgsConstructor;
+import org.apache.shardingsphere.infra.exception.ShardingSphereException;
+import org.apache.shardingsphere.infra.metadata.ShardingSphereMetaData;
+import org.apache.shardingsphere.sharding.route.engine.exception.NoSuchTableException;
+import org.apache.shardingsphere.sharding.route.engine.exception.TableExistsException;
+import org.apache.shardingsphere.sql.parser.sql.common.segment.generic.table.SimpleTableSegment;
+
+import java.util.Collection;
+import java.util.Map;
+
+/**
+ * Statement validator utility class.
+ */
+@NoArgsConstructor(access = AccessLevel.PRIVATE)
+public final class StatementValidatorUtil {

Review comment:
       > Is this class just for some of the unsupported DDLSQLStataments? In that case, do you think an abstract class to carry these functions will be better?
   > `StatementValidatorUtil` gives me a feeling that it is born for all the statements.
   
   @tristaZero I agree with your point, how about adding a new abstract class `ShardingDDLStatementValidator` to carry these functions？




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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [shardingsphere] tristaZero commented on a change in pull request #7740: support create, alter, drop, select view statement

Posted by GitBox <gi...@apache.org>.
tristaZero commented on a change in pull request #7740:
URL: https://github.com/apache/shardingsphere/pull/7740#discussion_r502900025



##########
File path: shardingsphere-features/shardingsphere-sharding/shardingsphere-sharding-route/src/main/java/org/apache/shardingsphere/sharding/route/engine/validator/util/StatementValidatorUtil.java
##########
@@ -0,0 +1,83 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.shardingsphere.sharding.route.engine.validator.util;
+
+import lombok.AccessLevel;
+import lombok.NoArgsConstructor;
+import org.apache.shardingsphere.infra.exception.ShardingSphereException;
+import org.apache.shardingsphere.infra.metadata.ShardingSphereMetaData;
+import org.apache.shardingsphere.sharding.route.engine.exception.NoSuchTableException;
+import org.apache.shardingsphere.sharding.route.engine.exception.TableExistsException;
+import org.apache.shardingsphere.sql.parser.sql.common.segment.generic.table.SimpleTableSegment;
+
+import java.util.Collection;
+import java.util.Map;
+
+/**
+ * Statement validator utility class.
+ */
+@NoArgsConstructor(access = AccessLevel.PRIVATE)
+public final class StatementValidatorUtil {

Review comment:
       Is this class just for some of the unsupported DDLSQLStataments?  In that case, do you think an abstract class to carry these functions will be better?
   `StatementValidatorUtil` gives me a  feeling that it is born for all the statements.




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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [shardingsphere] codecov-io commented on pull request #7740: support create, alter, drop, select view statement

Posted by GitBox <gi...@apache.org>.
codecov-io commented on pull request #7740:
URL: https://github.com/apache/shardingsphere/pull/7740#issuecomment-706653688


   # [Codecov](https://codecov.io/gh/apache/shardingsphere/pull/7740?src=pr&el=h1) Report
   > Merging [#7740](https://codecov.io/gh/apache/shardingsphere/pull/7740?src=pr&el=desc) into [master](https://codecov.io/gh/apache/shardingsphere/commit/e3bb018d4366da9d5f1ba51197ae3403a2596c99?el=desc) will **increase** coverage by `42.36%`.
   > The diff coverage is `69.14%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/shardingsphere/pull/7740/graphs/tree.svg?width=650&height=150&src=pr&token=ZvlXpWa7so)](https://codecov.io/gh/apache/shardingsphere/pull/7740?src=pr&el=tree)
   
   ```diff
   @@              Coverage Diff              @@
   ##             master    #7740       +/-   ##
   =============================================
   + Coverage     32.82%   75.19%   +42.36%     
   + Complexity     2381      521     -1860     
   =============================================
     Files          1417     1409        -8     
     Lines        104857    22471    -82386     
     Branches       9526     4042     -5484     
   =============================================
   - Hits          34415    16896    -17519     
   + Misses        67755     4485    -63270     
   + Partials       2687     1090     -1597     
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/shardingsphere/pull/7740?src=pr&el=tree) | Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | [.../route/engine/type/ShardingRouteEngineFactory.java](https://codecov.io/gh/apache/shardingsphere/pull/7740/diff?src=pr&el=tree#diff-c2hhcmRpbmdzcGhlcmUtZmVhdHVyZXMvc2hhcmRpbmdzcGhlcmUtc2hhcmRpbmcvc2hhcmRpbmdzcGhlcmUtc2hhcmRpbmctcm91dGUvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL3NoYXJkaW5nc3BoZXJlL3NoYXJkaW5nL3JvdXRlL2VuZ2luZS90eXBlL1NoYXJkaW5nUm91dGVFbmdpbmVGYWN0b3J5LmphdmE=) | `77.27% <0.00%> (-1.80%)` | `0.00 <0.00> (ø)` | |
   | [...sql/parser/mysql/visitor/impl/MySQLDDLVisitor.java](https://codecov.io/gh/apache/shardingsphere/pull/7740/diff?src=pr&el=tree#diff-c2hhcmRpbmdzcGhlcmUtc3FsLXBhcnNlci9zaGFyZGluZ3NwaGVyZS1zcWwtcGFyc2VyLWRpYWxlY3Qvc2hhcmRpbmdzcGhlcmUtc3FsLXBhcnNlci1teXNxbC9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvc2hhcmRpbmdzcGhlcmUvc3FsL3BhcnNlci9teXNxbC92aXNpdG9yL2ltcGwvTXlTUUxERExWaXNpdG9yLmphdmE=) | `47.82% <0.00%> (-0.89%)` | `0.00 <0.00> (ø)` | |
   | [.../postgresql/visitor/impl/PostgreSQLDDLVisitor.java](https://codecov.io/gh/apache/shardingsphere/pull/7740/diff?src=pr&el=tree#diff-c2hhcmRpbmdzcGhlcmUtc3FsLXBhcnNlci9zaGFyZGluZ3NwaGVyZS1zcWwtcGFyc2VyLWRpYWxlY3Qvc2hhcmRpbmdzcGhlcmUtc3FsLXBhcnNlci1wb3N0Z3Jlc3FsL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9zaGFyZGluZ3NwaGVyZS9zcWwvcGFyc2VyL3Bvc3RncmVzcWwvdmlzaXRvci9pbXBsL1Bvc3RncmVTUUxERExWaXNpdG9yLmphdmE=) | `93.70% <0.00%> (-0.75%)` | `0.00 <0.00> (ø)` | |
   | [...t/postgresql/ddl/PostgreSQLAlterViewStatement.java](https://codecov.io/gh/apache/shardingsphere/pull/7740/diff?src=pr&el=tree#diff-c2hhcmRpbmdzcGhlcmUtc3FsLXBhcnNlci9zaGFyZGluZ3NwaGVyZS1zcWwtcGFyc2VyLXN0YXRlbWVudC9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvc2hhcmRpbmdzcGhlcmUvc3FsL3BhcnNlci9zcWwvZGlhbGVjdC9zdGF0ZW1lbnQvcG9zdGdyZXNxbC9kZGwvUG9zdGdyZVNRTEFsdGVyVmlld1N0YXRlbWVudC5qYXZh) | `0.00% <0.00%> (ø)` | `0.00 <0.00> (?)` | |
   | [...e/validator/ShardingStatementValidatorFactory.java](https://codecov.io/gh/apache/shardingsphere/pull/7740/diff?src=pr&el=tree#diff-c2hhcmRpbmdzcGhlcmUtZmVhdHVyZXMvc2hhcmRpbmdzcGhlcmUtc2hhcmRpbmcvc2hhcmRpbmdzcGhlcmUtc2hhcmRpbmctcm91dGUvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL3NoYXJkaW5nc3BoZXJlL3NoYXJkaW5nL3JvdXRlL2VuZ2luZS92YWxpZGF0b3IvU2hhcmRpbmdTdGF0ZW1lbnRWYWxpZGF0b3JGYWN0b3J5LmphdmE=) | `17.39% <18.75%> (-5.69%)` | `0.00 <0.00> (ø)` | |
   | [...dialect/handler/ddl/AlterViewStatementHandler.java](https://codecov.io/gh/apache/shardingsphere/pull/7740/diff?src=pr&el=tree#diff-c2hhcmRpbmdzcGhlcmUtc3FsLXBhcnNlci9zaGFyZGluZ3NwaGVyZS1zcWwtcGFyc2VyLXN0YXRlbWVudC9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvc2hhcmRpbmdzcGhlcmUvc3FsL3BhcnNlci9zcWwvZGlhbGVjdC9oYW5kbGVyL2RkbC9BbHRlclZpZXdTdGF0ZW1lbnRIYW5kbGVyLmphdmE=) | `33.33% <33.33%> (ø)` | `0.00 <0.00> (?)` | |
   | [...ialect/handler/ddl/CreateViewStatementHandler.java](https://codecov.io/gh/apache/shardingsphere/pull/7740/diff?src=pr&el=tree#diff-c2hhcmRpbmdzcGhlcmUtc3FsLXBhcnNlci9zaGFyZGluZ3NwaGVyZS1zcWwtcGFyc2VyLXN0YXRlbWVudC9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvc2hhcmRpbmdzcGhlcmUvc3FsL3BhcnNlci9zcWwvZGlhbGVjdC9oYW5kbGVyL2RkbC9DcmVhdGVWaWV3U3RhdGVtZW50SGFuZGxlci5qYXZh) | `33.33% <33.33%> (ø)` | `0.00 <0.00> (?)` | |
   | [...ator/impl/ShardingAlterViewStatementValidator.java](https://codecov.io/gh/apache/shardingsphere/pull/7740/diff?src=pr&el=tree#diff-c2hhcmRpbmdzcGhlcmUtZmVhdHVyZXMvc2hhcmRpbmdzcGhlcmUtc2hhcmRpbmcvc2hhcmRpbmdzcGhlcmUtc2hhcmRpbmctcm91dGUvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL3NoYXJkaW5nc3BoZXJlL3NoYXJkaW5nL3JvdXRlL2VuZ2luZS92YWxpZGF0b3IvaW1wbC9TaGFyZGluZ0FsdGVyVmlld1N0YXRlbWVudFZhbGlkYXRvci5qYXZh) | `90.90% <90.90%> (ø)` | `0.00 <0.00> (?)` | |
   | [...tor/impl/ShardingCreateViewStatementValidator.java](https://codecov.io/gh/apache/shardingsphere/pull/7740/diff?src=pr&el=tree#diff-c2hhcmRpbmdzcGhlcmUtZmVhdHVyZXMvc2hhcmRpbmdzcGhlcmUtc2hhcmRpbmcvc2hhcmRpbmdzcGhlcmUtc2hhcmRpbmctcm91dGUvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL3NoYXJkaW5nc3BoZXJlL3NoYXJkaW5nL3JvdXRlL2VuZ2luZS92YWxpZGF0b3IvaW1wbC9TaGFyZGluZ0NyZWF0ZVZpZXdTdGF0ZW1lbnRWYWxpZGF0b3IuamF2YQ==) | `90.90% <90.90%> (ø)` | `0.00 <0.00> (?)` | |
   | [...impl/ShardingCreateFunctionStatementValidator.java](https://codecov.io/gh/apache/shardingsphere/pull/7740/diff?src=pr&el=tree#diff-c2hhcmRpbmdzcGhlcmUtZmVhdHVyZXMvc2hhcmRpbmdzcGhlcmUtc2hhcmRpbmcvc2hhcmRpbmdzcGhlcmUtc2hhcmRpbmctcm91dGUvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL3NoYXJkaW5nc3BoZXJlL3NoYXJkaW5nL3JvdXRlL2VuZ2luZS92YWxpZGF0b3IvaW1wbC9TaGFyZGluZ0NyZWF0ZUZ1bmN0aW9uU3RhdGVtZW50VmFsaWRhdG9yLmphdmE=) | `90.90% <100.00%> (-4.75%)` | `0.00 <0.00> (ø)` | |
   | ... and [73 more](https://codecov.io/gh/apache/shardingsphere/pull/7740/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/shardingsphere/pull/7740?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/shardingsphere/pull/7740?src=pr&el=footer). Last update [e3bb018...d40f65b](https://codecov.io/gh/apache/shardingsphere/pull/7740?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org