Remove excess commas from mysql-schema.sql

mroldx commented 4 days ago

Please do not create a Pull Request without creating an issue first.

What is the purpose of the change

Brief changelog

publish_type varchar(50) DEFAULT 'formal' COMMENT 'publish type gray or formal',, -----》
publish_type varchar(50) DEFAULT 'formal' COMMENT 'publish type gray or formal',

Verifying this change

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[ ] Run mvn -B clean package apache-rat:check findbugs:findbugs -Dmaven.test.skip=true to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test to make sure integration-test pass.

github-actions[bot] commented 4 days ago

Thanks for your this PR. :pray: Please check again for your PR changes whether contains any usage/api/configuration change such as Add new API, Add new configuration, Change default value of configuration. If so, please add or update documents(markdown type) in docs/next/ for repository nacos-group/nacos-group.github.io

感谢您提交的PR。 :pray: 请再次查看您的PR内容，确认是否包含任何使用方式/API/配置参数的变更，如：新增API、新增配置参数、修改默认配置等操作。如果是，请确保在提交之前，在仓库nacos-group/nacos-group.github.io中的docs/next/目录下添加或更新文档（markdown格式）。

codecov-commenter commented 4 days ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 72.16%. Comparing base (aeaaded) to head (e3069dc).

Additional details and impacted files

[![Impacted file tree graph](https://app.codecov.io/gh/alibaba/nacos/pull/12860/graphs/tree.svg?width=650&height=150&src=pr&token=l4XELZqush&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=alibaba)](https://app.codecov.io/gh/alibaba/nacos/pull/12860?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=alibaba) ```diff @@ Coverage Diff @@ ## develop #12860 +/- ## ============================================= - Coverage 72.17% 72.16% -0.02% Complexity 9792 9792 ============================================= Files 1283 1283 Lines 41387 41387 Branches 4374 4374 ============================================= - Hits 29872 29866 -6 - Misses 9408 9416 +8 + Partials 2107 2105 -2 ``` [see 3 files with indirect coverage changes](https://app.codecov.io/gh/alibaba/nacos/pull/12860/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=alibaba) ------ [Continue to review full report in Codecov by Sentry](https://app.codecov.io/gh/alibaba/nacos/pull/12860?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=alibaba). > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=alibaba) > `Δ = absolute (impact)`, `ø = not affected`, `? = missing data` > Powered by [Codecov](https://app.codecov.io/gh/alibaba/nacos/pull/12860?dropdown=coverage&src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=alibaba). Last update [aeaaded...e3069dc](https://app.codecov.io/gh/alibaba/nacos/pull/12860?dropdown=coverage&src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=alibaba). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=alibaba).

wuyfee commented 4 days ago

$\color{red}{FAILURE}$ DETAILS ✅ - docker: success ❌ - deploy (standalone & cluster & standalone_auth): failure ❌ - e2e-java-test (standalone & cluster & standalone_auth): skipped ❌ - e2e-go-test (standalone & cluster): skipped ❌ - e2e-cpp-test (standalone & cluster): skipped ❌ - e2e-csharp-test (standalone & cluster): skipped ❌ - e2e-nodejs-test (standalone & cluster): skipped ❌ - e2e-python-test (standalone & cluster): skipped ✅ - clean (standalone & cluster & standalone_auth): success

KomachiSion commented 1 day ago

There are some e2e error in this pr and PR #12866 has fixed this problem.

alibaba / nacos