Skip to content

[type: bug] fix h2 sql#3837

Merged
loongs-zhang merged 1 commit intoapache:masterfrom
yu199195:fix-sql
Aug 12, 2022
Merged

[type: bug] fix h2 sql#3837
loongs-zhang merged 1 commit intoapache:masterfrom
yu199195:fix-sql

Conversation

@yu199195
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.

@yu199195 yu199195 added type: bug Something isn't working admin labels Aug 12, 2022
@yu199195 yu199195 added this to the 2.5.0 milestone Aug 12, 2022
@codecov-commenter
Copy link

codecov-commenter commented Aug 12, 2022

Codecov Report

Merging #3837 (426d5b0) into master (db33237) will increase coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3837      +/-   ##
============================================
+ Coverage     64.74%   64.76%   +0.01%     
+ Complexity     6099     6097       -2     
============================================
  Files           898      898              
  Lines         24708    24708              
  Branches       2240     2240              
============================================
+ Hits          15998    16002       +4     
+ Misses         7304     7300       -4     
  Partials       1406     1406              
Impacted Files Coverage Δ
...va/org/apache/shenyu/common/dto/ConditionData.java 68.18% <ø> (ø)
...ruptor/RegisterClientServerDisruptorPublisher.java 64.70% <0.00%> (+11.76%) ⬆️
...controller/ShenyuClientHttpRegistryController.java 100.00% <0.00%> (+22.22%) ⬆️

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

@loongs-zhang loongs-zhang merged commit 6a08b6c into apache:master Aug 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

admin type: bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants