sandynz opened a new pull request, #20860: URL: https://github.com/apache/shardingsphere/pull/20860
Changes proposed in this pull request: - Register default SPI impl - Refactor SPI factory to compatable with TypedSPI and RequiredSPI - Add DefaultPositionInitializer - Refactor FactoryTest as real test - Clean some fixture SPI impl that's not necessary --- Before committing this PR, I'm sure that I have checked the following options: - [x] My code follows the [code of conduct](https://shardingsphere.apache.org/community/en/involved/conduct/code/) of this project. - [x] I have self-reviewed the commit code. - [x] I have triggered maven check: `mvn clean install -B -T2C -DskipTests -Dmaven.javadoc.skip=true -e`. - [x] I have made corresponding changes to the documentation. - [x] I have added corresponding unit tests for my changes. -- 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]
