-
Notifications
You must be signed in to change notification settings - Fork 13.2k
feat(plugin): Added support for the xugu database #14038
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: develop
Are you sure you want to change the base?
Conversation
- new xugu database Mapper implementation classes, including ConfigInfoBetaMapperByXugu, ConfigInfoGrayMapperByXugu, etc The abstract Mapper class AbstractMapperByXugu for the xugu database has been implemented Add an example of xugu database connection configuration in the bootstrap configuration file Update the Mapper service registration file and add the Mapper implementation related to xugu Extend the DataSourceConstant constant class and add the XU_GU data source identifier - Implement GroupCapacityMapper for the xugu database and related SQL adaptations Implement ConfigInfoMapper and pagination query logic for the xugu database - Add the xugu version implementation of ConfigTagsRelationMapper - Complete xugu support for ConfigMigrateMapper and ConfigInfoTagMapper
This reverts commit e0e837e.
- new xugu database Mapper implementation classes, including ConfigInfoBetaMapperByXugu, ConfigInfoGrayMapperByXugu, etc The abstract Mapper class AbstractMapperByXugu for the xugu database has been implemented Add an example of xugu database connection configuration in the bootstrap configuration file Update the Mapper service registration file and add the Mapper implementation related to xugu Extend the DataSourceConstant constant class and add the XU_GU data source identifier - Implement GroupCapacityMapper for the xugu database and related SQL adaptations Implement ConfigInfoMapper and pagination query logic for the xugu database - Add the xugu version implementation of ConfigTagsRelationMapper - Complete xugu support for ConfigMigrateMapper and ConfigInfoTagMapper
|
Thanks for your this PR. 🙏 感谢您提交的PR。 🙏 |
|
Thank you for your help! Let's review it. @KomachiSion |
|
Please submit new plugin into nacos-group/nacos-plugin. Thanks. |
Ok! Thank you for the reminder. |
But I noticed that this plugin seems to only support nacos2.4.3? |
# Conflicts: # plugin-default-impl/nacos-default-datasource-plugin/src/main/resources/META-INF/services/com.alibaba.nacos.plugin.datasource.mapper.Mapper # pom.xml
|
jowee seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
|
You can add description in README.md in nacos-group/nacos-plugin to tell users xugu plugin only support after version xxx. |
The abstract Mapper class AbstractMapperByXugu for the xugu database has been implemented
Add an example of xugu database connection configuration in the bootstrap configuration file
Update the Mapper service registration file and add the Mapper implementation related to xugu
Extend the DataSourceConstant constant class and add the XU_GU data source identifier
Implement ConfigInfoMapper and pagination query logic for the xugu database
Add the xugu version implementation of ConfigTagsRelationMapper
Complete xugu support for ConfigMigrateMapper and ConfigInfoTagMapper
Please do not create a Pull Request without creating an issue first.
What is the purpose of the change
XXXXX
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily:
[ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.mvn -B clean package apache-rat:check findbugs:findbugs -Dmaven.test.skip=trueto make sure basic checks pass. Runmvn clean install -DskipITsto make sure unit-test pass. Runmvn clean test-compile failsafe:integration-testto make sure integration-test pass.