fix: BinlogDeserializeReader leak

[fengjun2016]

#36086

[codecov]

Codecov Report

Attention: Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 81.58%. Comparing base (7859faf) to head (218eaaa).
Report is 24 commits behind head on master.

Files with missing lines	Patch %	Lines
internal/indexnode/task_stats.go	50.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master   #36087      +/-   ##
==========================================
+ Coverage   72.62%   81.58%   +8.96%     
==========================================
  Files        1266     1266              
  Lines      150757   150760       +3     
==========================================
+ Hits       109485   123004   +13519     
+ Misses      36395    22880   -13515     
+ Partials     4877     4876       -1     

Files with missing lines	Coverage Δ
internal/indexnode/task_stats.go	77.03% <50.00%> (+0.69%)	⬆️

... and 241 files with indirect coverage changes

[xiaofan-luan]

/lgtm
/approve

[xiaofan-luan]

good catch! man you are so good at it!

[sre-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: fengjun2016, xiaofan-luan

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• internal/indexnode/OWNERS [xiaofan-luan]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[fengjun2016]

good catch! man you are so good at it!

Thank you for your compliment.
It’s just that in my daily work, I have accumulated a little sensitivity in the process of code review and a little bit of mysophobia about the code.
Of course, too much mysophobia about the code is not always a good thing.