lalicw opened a new issue #1881: ShardingStrategyConfiguration changed, and  
unfriendly to extension
URL: https://github.com/apache/incubator-shardingsphere/issues/1881
 
 
   in the old version (2.x) of sharding-jdbc, the ShardingStrategyConfiguration 
interface has a build method.
   so we can make extension like CustomStrategyConfiguration and use 
CustomStrategy extend ShardingStrategy.
   
   now in the new version (3.x),  it use a final class ShardingStrategyFactory 
to create ShardingStrategy and we cannot extend it anymore. It may be a bad 
idea to do this change and customized sharding strategy is necessary sometimes.
   
   Otherwise we suggest to add some context to ShardingStrategy.doSharding 
method, because we need more info to do the sharding, not only the 
availableTargetNames and shardingValues.
   
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to