-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
[ISSUE #12197]Add an id primary key column to both the roles and permissions tables… #12206
Conversation
….(mysql-schema.sql)
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #12206 +/- ##
=============================================
+ Coverage 68.76% 68.81% +0.05%
- Complexity 9227 9233 +6
=============================================
Files 1264 1264
Lines 41183 41187 +4
Branches 4361 4363 +2
=============================================
+ Hits 28318 28344 +26
+ Misses 10828 10806 -22
Partials 2037 2037 see 8 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
…s tables.(mysql-schema.sql) (alibaba#12206)" This reverts commit 5d87196.
…apexsoft * commit '197795a854992d3f88824e4cdde34c61d8f56cf4': (225 commits) Fix alibaba#12395, use request context replace session depend. (alibaba#12398) Revert "Add an id primary key column to both the roles and permissions tables.(mysql-schema.sql) (alibaba#12206)" (alibaba#12396) Upgrade to 2.4.0 (alibaba#12384) Develop support ram info switch (alibaba#12382) Close derby ops api default. (alibaba#12372) Upgrade grpc to 1.64.2 (alibaba#12369) fix(alibaba#12333): fixed auth Plugin resource parser can't parser v2 config openAPI namespaceId. (alibaba#12336) Adjust integration tests for common modules in the core module to comply with checkstyle. (alibaba#12349) [IT]Adjust integration tests for cluster,code,smoke modules in the core module to comply with checkstyle. (alibaba#12350) fix [ISSUE alibaba#12323] , using the project's logback packagingData configuration instead of logback-nacos.xml's packagingData configuration (alibaba#12360) Remove the accessToken from the URL. (alibaba#12353) feat(alibaba#12355): Record the cost of ConfigDump in Prometheus. (alibaba#12356) [IT]Adjust core module auth integration tests to meet checkstyle requirements. (alibaba#12368) [ISSUE alibaba#12282]Fix the issue where monitoring data cannot be found through the IP dimension in the ListeningQuery (alibaba#12315) Adjust integration tests for ability, base, and client modules in the core module to comply with checkstyle. (alibaba#12335) Develop refactor request context (alibaba#12331) fix unexpect exception from NetworkInterface.ifUp (alibaba#12325) fix(alibaba#12231): When inserting and updating configurations in the database, the time-related field values need to be set using the time obtained from the database's built-in time function. (alibaba#12244) add some UT for default auth plugin (alibaba#12318) [ISSUE alibaba#12219] 修复nacos-client中config的ServerListManager解析endpoint失败问题 (alibaba#12220) ... # Conflicts: # plugin-default-impl/nacos-default-auth-plugin/src/main/java/com/alibaba/nacos/plugin/auth/impl/persistence/ExternalPermissionPersistServiceImpl.java # pom.xml
What is the purpose of the change
fix #12197
Add an id primary key column to both the roles and permissions tables.(mysql-schema.sql
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=true
to make sure basic checks pass. Runmvn clean install -DskipITs
to make sure unit-test pass. Runmvn clean test-compile failsafe:integration-test
to make sure integration-test pass.