[#133] feat(netty): Fix IllegalReferenceCountException.

[leixm]

What changes were proposed in this pull request?

The ByteBuf.release() of the block should be called after clearInFlushBuffer is called, otherwise there will be a certain probability of IllegalReferenceCountException at ShuffleBuffer#updateShuffleData -> data.addComponent(true, block.getData().retain());
1.Fix IllegalReferenceCountException,
2.Added ByteBuf.release() when remove resources.

Why are the changes needed?

Fix: #133

Does this PR introduce any user-facing change?

No.

How was this patch tested?

existing UTs.

[leixm]

@smallzhongfeng @jerqi PTAL.

[codecov-commenter]

Codecov Report

Merging #899 (0d461d0) into master (9c62784) will increase coverage by 1.87%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #899      +/-   ##
============================================
+ Coverage     55.22%   57.09%   +1.87%     
- Complexity     2197     2199       +2     
============================================
  Files           333      313      -20     
  Lines         16444    14087    -2357     
  Branches       1306     1306              
============================================
- Hits           9081     8043    -1038     
+ Misses         6851     5603    -1248     
+ Partials        512      441      -71     

Impacted Files	Coverage Δ
...le/storage/handler/impl/LocalFileWriteHandler.java	77.08% <ø> (-0.47%)	⬇️
...rg/apache/uniffle/server/buffer/ShuffleBuffer.java	92.74% <100.00%> (+0.17%)	⬆️
...he/uniffle/server/buffer/ShuffleBufferManager.java	84.28% <100.00%> (+0.05%)	⬆️

... and 21 files with indirect coverage changes

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

[jerqi]

LGTM, thanks.

[leixm]

@jerqi Thank you for your review.

[leixm]

Seems WriteAndReadMetricsTest is a flaky test.