bugfix: fix potential database resource leak

[slievrly]

Signed-off-by: slievrly slievrly@163.com

• I have registered the PR changes.

Ⅰ. Describe what this PR did

bugfix: fix potential database resource leak

Ⅱ. Does this pull request fix one issue?

fix #4024

Ⅲ. Why don't you add test cases (unit test/integration test)?

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

[codecov-commenter]

Codecov Report

Merging #4025 (fa8eed4) into develop (8456fb6) will increase coverage by 0.00%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##             develop    #4025   +/-   ##
==========================================
  Coverage      49.91%   49.92%           
+ Complexity      3767     3766    -1     
==========================================
  Files            698      698           
  Lines          23509    23515    +6     
  Branches        2910     2910           
==========================================
+ Hits           11735    11739    +4     
- Misses         10593    10594    +1     
- Partials        1181     1182    +1     

Impacted Files	Coverage Δ
...ommon/loader/EnhancedServiceNotFoundException.java	27.27% <ø> (ø)
...ery/registry/consul/ConsulRegistryServiceImpl.java	1.70% <0.00%> (ø)
...scovery/loadbalance/ConsistentHashLoadBalance.java	89.28% <ø> (ø)
...o/seata/integration/http/AbstractHttpExecutor.java	64.15% <ø> (ø)
...o/seata/rm/datasource/exec/BaseInsertExecutor.java	71.12% <ø> (ø)
...tatelang/parser/utils/DesignerJsonTransformer.java	54.65% <0.00%> (ø)
...java/io/seata/tm/api/DefaultGlobalTransaction.java	65.09% <ø> (ø)
...source/exec/mysql/MySQLInsertOrUpdateExecutor.java	31.42% <50.00%> (ø)
.../rm/datasource/exec/mysql/MySQLInsertExecutor.java	70.00% <71.42%> (+0.58%)	⬆️
...seata/rm/datasource/undo/AbstractUndoExecutor.java	67.14% <100.00%> (+0.23%)	⬆️
... and 1 more

[lgtm-com]

This pull request fixes 5 alerts when merging 511bde7 into b71ba74 - view on LGTM.com

fixed alerts:

• 2 for Potential database resource leak
• 2 for Dereferenced variable may be null
• 1 for Non-synchronized override of synchronized method

[funky-eyes]

LGTM

[lgtm-com]

This pull request fixes 5 alerts when merging fa8eed4 into 8456fb6 - view on LGTM.com

fixed alerts:

• 2 for Potential database resource leak
• 2 for Dereferenced variable may be null
• 1 for Non-synchronized override of synchronized method