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:
[email protected]


Reply via email to