likly commented on pull request #8587: URL: https://github.com/apache/shardingsphere/pull/8587#issuecomment-743580420
> Sharding algorithms are dynamic and loaded via SPI. > The enum is static config and broken OCP. > So it is better to do not use enum to define sharding algorithm types. This Enum only used for build-in sharding algorithms to manage the SPI types easily. And this enum is not broken OCP, Sharding algorithms still are dynamic and loaded via SPI. ---------------------------------------------------------------- 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]
