Skip to content

Pipe: fix file handle leak when processing with extractor.pattern#11209

Merged
SteveYurongSu merged 1 commit intomasterfrom
fix-pipe-file-handle-leak
Sep 25, 2023
Merged

Pipe: fix file handle leak when processing with extractor.pattern#11209
SteveYurongSu merged 1 commit intomasterfrom
fix-pipe-file-handle-leak

Conversation

@SteveYurongSu
Copy link
Copy Markdown
Member

As title.

@sonarqubecloud
Copy link
Copy Markdown

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
22.2% 22.2% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #11209 (32b6f3a) into master (3250550) will decrease coverage by 0.01%.
The diff coverage is 4.54%.

@@             Coverage Diff              @@
##             master   #11209      +/-   ##
============================================
- Coverage     47.58%   47.58%   -0.01%     
- Complexity    23671    23672       +1     
============================================
  Files          2789     2789              
  Lines        170528   170538      +10     
  Branches      20318    20318              
============================================
+ Hits          81152    81156       +4     
- Misses        89376    89382       +6     
Files Changed Coverage Δ
...onnector/protocol/airgap/IoTDBAirGapConnector.java 0.00% <0.00%> (ø)
...otocol/thrift/async/IoTDBThriftAsyncConnector.java 8.78% <0.00%> (-0.04%) ⬇️
...nnector/protocol/websocket/WebSocketConnector.java 0.00% <0.00%> (ø)
.../event/common/tsfile/PipeTsFileInsertionEvent.java 29.48% <0.00%> (-2.46%) ⬇️
...nt/common/tsfile/TsFileInsertionDataContainer.java 86.95% <100.00%> (+0.14%) ⬆️

... and 13 files with indirect coverage changes

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

@SteveYurongSu SteveYurongSu merged commit 3f8c344 into master Sep 25, 2023
@SteveYurongSu SteveYurongSu deleted the fix-pipe-file-handle-leak branch September 25, 2023 08:56
SteveYurongSu added a commit that referenced this pull request Sep 25, 2023
SteveYurongSu added a commit that referenced this pull request Sep 25, 2023
HTHou added a commit that referenced this pull request Sep 26, 2023
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.

2 participants