Skip to content

Conversation

@TeslaCN
Copy link
Member

@TeslaCN TeslaCN commented Aug 31, 2022

Fixes #20667.

@codecov-commenter
Copy link

Codecov Report

Merging #20669 (a77de46) into master (38ea308) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master   #20669      +/-   ##
============================================
+ Coverage     61.07%   61.08%   +0.01%     
  Complexity     2414     2414              
============================================
  Files          3956     3956              
  Lines         54898    54885      -13     
  Branches       9313     9312       -1     
============================================
+ Hits          33527    33528       +1     
+ Misses        18520    18506      -14     
  Partials       2851     2851              
Impacted Files Coverage Δ
...ore/statement/ShardingSpherePreparedStatement.java 65.29% <100.00%> (+1.29%) ⬆️
...r/jdbc/core/statement/ShardingSphereStatement.java 52.26% <100.00%> (+1.74%) ⬆️

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

@TeslaCN TeslaCN marked this pull request as ready for review August 31, 2022 05:44
@tristaZero tristaZero merged commit c4bc33d into apache:master Aug 31, 2022
@TeslaCN TeslaCN deleted the 20667 branch August 31, 2022 05:46
@TeslaCN TeslaCN added this to the 5.2.1 milestone Aug 31, 2022
lhyundeadsoul pushed a commit to lhyundeadsoul/shardingsphere that referenced this pull request Sep 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ShardingSphere-JDBC throws exceptions which is not SQLException (or its sub-class)

3 participants