Skip to content

[ISSUE #5047] Fix thread leak in TCP plugin#5048

Merged
moremind merged 2 commits intoapache:masterfrom
TeslaCN:fix-tcp-thread-leak
Aug 19, 2023
Merged

[ISSUE #5047] Fix thread leak in TCP plugin#5048
moremind merged 2 commits intoapache:masterfrom
TeslaCN:fix-tcp-thread-leak

Conversation

@TeslaCN
Copy link
Member

@TeslaCN TeslaCN commented Aug 18, 2023

Fixes #5047.

Make sure that:

  • You have read the contribution guidelines.
  • You submit test cases (unit or integration tests) that back your changes.
  • Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

@TeslaCN TeslaCN marked this pull request as ready for review August 18, 2023 14:25
@TeslaCN TeslaCN force-pushed the fix-tcp-thread-leak branch from 4b1a877 to ac59d35 Compare August 18, 2023 14:38
@codecov-commenter
Copy link

codecov-commenter commented Aug 19, 2023

Codecov Report

Merging #5048 (2394653) into master (483902c) will not change coverage.
Report is 1 commits behind head on master.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #5048   +/-   ##
=========================================
  Coverage     64.06%   64.06%           
+ Complexity     8507     8496   -11     
=========================================
  Files          1208     1208           
  Lines         35540    35540           
  Branches       3206     3206           
=========================================
  Hits          22770    22770           
+ Misses        10923    10921    -2     
- Partials       1847     1849    +2     

see 6 files with indirect coverage changes

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

@moremind moremind requested a review from ywj1352 August 19, 2023 04:43
@moremind moremind merged commit 38e3c6a into apache:master Aug 19, 2023
@TeslaCN TeslaCN deleted the fix-tcp-thread-leak branch August 19, 2023 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] EventLoop threads leak in TCP plugin

3 participants