strongduanmu opened a new issue #13441:
URL: https://github.com/apache/shardingsphere/issues/13441


   ## Background
   
   This issue is a problem encountered during the issue #13091. In order to 
route accurately for federation executor and hide the actual tables and actual 
columns, we need to move the route engine, rewrite engine, executor, and merge 
engine into the federation execution engine, which requires us to add 
dependencies to the federation execution engine.
   
   The current dependencies are as follows. If we add `infra-merge` dependency 
in the infra-executor, it will cause a circular dependency.
   
   ```
   
org.apache.shardingsphere:shardingsphere-infra-executor:jar:5.0.0-RC1-SNAPSHOT
   
|----org.apache.shardingsphere:shardingsphere-infra-common:jar:5.0.0-RC1-SNAPSHOT:compile
   
|----org.apache.shardingsphere:shardingsphere-infra-rewrite:jar:5.0.0-RC1-SNAPSHOT:compile
   
|----org.apache.shardingsphere:shardingsphere-infra-optimize:jar:5.0.0-RC1-SNAPSHOT:compile
   |----org.apache.calcite:calcite-core:jar:1.27.0:compile
   
|----org.apache.shardingsphere:shardingsphere-sql-parser-mysql:jar:5.0.0-RC1-SNAPSHOT:test
   
   org.apache.shardingsphere:shardingsphere-infra-merge:jar:5.0.0-RC1-SNAPSHOT
   
|----org.apache.shardingsphere:shardingsphere-infra-binder:jar:5.0.0-RC1-SNAPSHOT:compile
   
|----org.apache.shardingsphere:shardingsphere-infra-executor:jar:5.0.0-RC1-SNAPSHOT:compile
   ```
   
   ## Adjust dependencies
   
   So I want to discuss whether `infra-executor` can be split into multiple 
sub-modules, the structure is as follows:
   
   ```
   shardingsphere-infra-executor
   |----shardingsphere-infra-executor-driver
   |----shardingsphere-infra-executor-federation
        |----shardingsphere-infra-context
        |----shardingsphere-infra-optimize
        |----shardingsphere-infra-merge
   ```
   
   The `shardingsphere-infra-executor-driver` module is responsible for 
handling standard kernel execution. The 
`shardingsphere-infra-executor-federation` module is responsible for handling 
query optimization statement execution.
   
   Existing modules that rely on `shardingsphere-infra-executor` can be 
replaced with `shardingsphere-infra-executor-driver`. If there are some modules 
that rely on `shardingsphere-infra-executor-federation`, just add dependencies.
   


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