guimingyue commented on a change in pull request #9934:
URL: https://github.com/apache/shardingsphere/pull/9934#discussion_r607599406



##########
File path: shardingsphere-infra/pom.xml
##########
@@ -40,5 +40,6 @@
         <module>shardingsphere-infra-context</module>
         <module>shardingsphere-infra-datetime</module>
         <module>shardingsphere-infra-optimize</module>
+        <module>shardingsphere-infra-optimizer</module>

Review comment:
       The module `shardingsphere-infra-optimize` can be regarded as an adaptor 
of calcite, and cannot optimize the join operator which can be pushdown to the 
sharding database. `shardingsphere-infra-optimizer` module does not depend on 
the `shardingsphere-infra-executor` which is different from 
`shardingsphere-infra-optimize` too. The `shardingsphere-infra-executor` will 
depend on the `shardingsphere-infra-optimizer` for the execution of the 
physical plan.




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