optimize: auto closeable

[jsbxyyx]

Ⅰ. Describe what this PR did

optimize: auto closeable

Ⅱ. Does this pull request fix one issue?

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

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

[codecov-io]

Codecov Report

Merging #1884 into develop will increase coverage by 0.18%.
The diff coverage is 78.12%.

@@              Coverage Diff              @@
##             develop    #1884      +/-   ##
=============================================
+ Coverage      55.16%   55.34%   +0.18%     
- Complexity      2377     2382       +5     
=============================================
  Files            422      423       +1     
  Lines          14395    14270     -125     
  Branches        1730     1693      -37     
=============================================
- Hits            7941     7898      -43     
+ Misses          5709     5662      -47     
+ Partials         745      710      -35

Impacted Files	Coverage Δ	Complexity Δ
...c/main/java/io/seata/compressor/gzip/GzipUtil.java	88.88% <100%> (+10.1%)	5 <0> (ø)	⬇️
...va/io/seata/core/store/db/LogStoreDataBaseDAO.java	57.48% <100%> (+3%)	21 <0> (ø)	⬇️
...c/main/java/io/seata/common/util/CompressUtil.java	96% <100%> (-0.43%)	7 <0> (ø)
.../io/seata/common/loader/EnhancedServiceLoader.java	63.02% <100%> (+0.73%)	24 <0> (ø)	⬇️
...seata/rm/datasource/undo/AbstractUndoExecutor.java	76.41% <100%> (+2.07%)	21 <0> (ø)	⬇️
...a/io/seata/core/store/db/LockStoreDataBaseDAO.java	50.6% <50%> (+4.17%)	18 <0> (ø)	⬇️
...mon/src/main/java/io/seata/common/util/IOUtil.java	60% <60%> (ø)	3 <3> (?)
...o/seata/server/coordinator/DefaultCoordinator.java	48.41% <0%> (+0.39%)	28% <0%> (+1%)	⬆️
...in/java/io/seata/server/session/GlobalSession.java	85.02% <0%> (+0.48%)	68% <0%> (+1%)	⬆️
... and 1 more

[echooymxq]

why not use try-with-resources?

[helloworlde]

I found all resource in this change could use try-with-resource, and that can be closed automatic. Maybe change to use try-with-resource is better.

[jsbxyyx]

@echooymxq @helloworlde Not all places can be used try-with-resources.

[slievrly]

LGTM

[slievrly]

@jsbxyyx database operation related variable names can be optimized.

[jsbxyyx]

@slievrly OK.

[helloworlde]

@echooymxq @helloworlde Not all places can be used try-with-resources.

It's can be closed by multiple try.

[jsbxyyx]

@echooymxq @helloworlde Not all places can be used try-with-resources.

It's can be closed by multiple try.

If use try-with-resources, the conn close can not has other logic.

[xingfudeshi]

LGTM.