[GitHub] [shardingsphere] Ahoo-Wang opened a new issue, #17159: The `CosIdModShardingAlgorithm` lost the `mod` configuration parameter.

[GitBox <gi...@apache.org>]

Ahoo-Wang opened a new issue, #17159:
URL: https://github.com/apache/shardingsphere/issues/17159

   ## Bug Report
   
   **For English only**, other languages will not accept.
   
   Before report a bug, make sure you have:
   
   - Searched open and closed [GitHub issues](https://github.com/apache/shardingsphere/issues).
   - Read documentation: [ShardingSphere Doc](https://shardingsphere.apache.org/document/current/en/overview).
   
   Please pay attention on issues you submitted, because we maybe need more details. 
   If no response anymore and we cannot reproduce it on current information, we will **close it**.
   
   Please answer these questions before submitting your issue. Thanks!
   
   ### Which version of ShardingSphere did you use?
   
   v5.1.0
   
   ### Which project did you use? ShardingSphere-JDBC or ShardingSphere-Proxy?
   
   ShardingSphere-JDBC
   
   ### Expected behavior
   
   The `CosIdModShardingAlgorithm` do not lost the `mod` configuration parameter.
   
   ### Actual behavior
   
   The `CosIdModShardingAlgorithm` lost the `mod` configuration parameter.
   
   ### Reason analyze (If you can)
   
   When the configuration parameter value is not of type `String`, using `Properties.getProperty` will return null even if `key` exists.
   
   Following is the implementation of `Properties.getProperty` method：
   
   ```java
       public String getProperty(String key) {
           Object oval = map.get(key);
           String sval = (oval instanceof String) ? (String)oval : null;  // the problem is here
           Properties defaults;
           return ((sval == null) && ((defaults = this.defaults) != null)) ? defaults.getProperty(key) : sval;
       }
   ```
   
   ### Steps to reproduce the behavior, such as: SQL to execute, sharding rule configuration, when exception occur etc.
   
   ### Example codes for reproduce this issue (such as a github link).
   


-- 
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: notifications-unsubscribe@shardingsphere.apache.org.apache.org

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

[GitHub] [shardingsphere] terrymanu closed issue #17159: The `CosIdModShardingAlgorithm` lost the `mod` configuration parameter.

[GitBox <gi...@apache.org>]

terrymanu closed issue #17159: The `CosIdModShardingAlgorithm` lost the `mod` configuration parameter.
URL: https://github.com/apache/shardingsphere/issues/17159


-- 
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: notifications-unsubscribe@shardingsphere.apache.org

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