menghaoranss opened a new issue #12289:
URL: https://github.com/apache/shardingsphere/issues/12289


   ## Question
   
   **For English only**, other languages will not accept.
   
   refer to #12195 
   
   In the class `SchemaRulePersistService`, the following method will check 
whether the rule configurations are complete before persisted.
   
   For ShardingSphere-Proxy, it could run successfully without any rules,  so 
this is unnecessary logic for proxy. 
   
   For ShardingSphere-JDBC, it should be checked before creating 
`ShardingSphereDataSource`
   
   
   ```
   private Collection<YamlRuleConfiguration> createYamlRuleConfigurations(final 
String schemaName, final Collection<RuleConfiguration> ruleConfigs) {
           Collection<RuleConfiguration> configs = new LinkedList<>();
           for (RuleConfiguration each : ruleConfigs) {
               
RuleConfigurationCheckerFactory.newInstance(each).ifPresent(checker -> 
checker.check(schemaName, each));
               configs.add(each);
           }
           return new 
YamlRuleConfigurationSwapperEngine().swapToYamlRuleConfigurations(configs);
       }
   ```
   
   Task list:
   
   - [ ] Remove checker in `SchemaRulePersistService` to fix #12195 
   - [ ] Add check in ShardingSphere-JDBC


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