search

housepower / ClickHouse-Native-JDBC

ClickHouse Native Protocol JDBC implementation
https://housepower.github.io/ClickHouse-Native-JDBC/
Apache License 2.0
527 stars 145 forks source link

fixbug/#432: The insert exception also sets the state to idle #435

Closed li-keguo closed 1 year ago

li-keguo commented 1 year ago

fixbug about issue ：#432

sundy-li commented 1 year ago

Thanks for the contribution, got lint error:

Error:  /home/runner/work/ClickHouse-Native-JDBC/ClickHouse-Native-JDBC/clickhouse-native-jdbc/src/main/java/com/github/housepower/jdbc/ClickHouseConnection.java:294:12: '{' is not preceded with whitespace. [WhitespaceAround]
codecov[bot] commented 1 year ago

Codecov Report

Merging #435 (3af24c3) into master (ffd0a65) will not change coverage. The diff coverage is 100.00%.

:exclamation: Current head 3af24c3 differs from pull request most recent head 73e6b63. Consider uploading reports for the commit 73e6b63 to get more accurate results

@@            Coverage Diff            @@
##             master     #435   +/-   ##
=========================================
  Coverage     62.99%   62.99%           
  Complexity     1263     1263           
=========================================
  Files           135      135           
  Lines          6662     6662           
  Branches        519      519           
=========================================
  Hits           4197     4197           
  Misses         2197     2197           
  Partials        268      268
Impacted Files Coverage Δ
...m/github/housepower/jdbc/ClickHouseConnection.java 65.35% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.