Skip to content
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

[type:fix] fix sql script error #4412

Merged
merged 2 commits into from
Feb 27, 2023

Conversation

moremind
Copy link
Member

Make sure that:

  • You have read the contribution guidelines.
  • You submit test cases (unit or integration tests) that back your changes.
  • Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

@codecov-commenter
Copy link

codecov-commenter commented Feb 27, 2023

Codecov Report

Merging #4412 (1288f58) into master (a842a3b) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #4412      +/-   ##
============================================
- Coverage     68.12%   68.09%   -0.04%     
+ Complexity     7486     7482       -4     
============================================
  Files          1021     1021              
  Lines         28913    28913              
  Branches       2584     2584              
============================================
- Hits          19697    19688       -9     
- Misses         7674     7682       +8     
- Partials       1542     1543       +1     
Impacted Files Coverage Δ
...controller/ShenyuClientHttpRegistryController.java 63.63% <0.00%> (-18.19%) ⬇️
...ruptor/RegisterClientServerDisruptorPublisher.java 55.55% <0.00%> (-11.12%) ⬇️
...henyu/plugin/grpc/resolver/ShenyuNameResolver.java 56.38% <0.00%> (-5.32%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@yu199195 yu199195 merged commit 4908790 into apache:master Feb 27, 2023
@yu199195 yu199195 added admin sql about sql changed labels Feb 27, 2023
@yu199195 yu199195 added this to the 2.6.0 milestone Feb 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
admin sql about sql changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants