Skip to content

[To dev/1.3] Pipe: Fix pipe executor stuck by unlimited file event memory allocation retries & Pipe: Fix IO triggered in disruptor & Pipe: Fix forceAllocateIfSufficient & Load: Remove RM detect logic during the second phase#15085

Merged
SteveYurongSu merged 11 commits intodev/1.3from
fix-pipe-stuck-by-file
Mar 14, 2025

Conversation

@SteveYurongSu
Copy link
Copy Markdown
Member

No description provided.

@SteveYurongSu SteveYurongSu self-assigned this Mar 13, 2025
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 13, 2025

Codecov Report

Attention: Patch coverage is 18.51852% with 22 lines in your changes missing coverage. Please review.

Project coverage is 41.42%. Comparing base (6efd4d9) to head (3e43a34).
Report is 4 commits behind head on dev/1.3.

Files with missing lines Patch % Lines
.../plan/scheduler/load/LoadTsFileDispatcherImpl.java 0.00% 8 Missing ⚠️
...ol/thrift/impl/DataNodeInternalRPCServiceImpl.java 0.00% 5 Missing ⚠️
...hrift/async/handler/PipeTransferTsFileHandler.java 0.00% 3 Missing ⚠️
...tdb/db/pipe/resource/memory/PipeMemoryManager.java 25.00% 3 Missing ⚠️
.../event/common/tsfile/PipeTsFileInsertionEvent.java 0.00% 1 Missing ⚠️
...ngine/plan/scheduler/load/LoadTsFileScheduler.java 0.00% 1 Missing ⚠️
...geengine/dataregion/wal/utils/WALEntryHandler.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##             dev/1.3   #15085      +/-   ##
=============================================
- Coverage      41.43%   41.42%   -0.01%     
  Complexity       193      193              
=============================================
  Files           3544     3544              
  Lines         228136   228136              
  Branches       27286    27287       +1     
=============================================
- Hits           94520    94504      -16     
- Misses        133616   133632      +16     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@SteveYurongSu SteveYurongSu changed the title [To dev/1.3] Pipe: Fix pipe executor stuck by unlimited file event memory allocation retries & Fix IO triggered in disruptor [To dev/1.3] Pipe: Fix pipe executor stuck by unlimited file event memory allocation retries & Pipe: Fix IO triggered in disruptor & Load: Remove RM detect logic during the second phase Mar 13, 2025
@SteveYurongSu SteveYurongSu changed the title [To dev/1.3] Pipe: Fix pipe executor stuck by unlimited file event memory allocation retries & Pipe: Fix IO triggered in disruptor & Load: Remove RM detect logic during the second phase [To dev/1.3] Pipe: Fix pipe executor stuck by unlimited file event memory allocation retries & Pipe: Fix IO triggered in disruptor & Pipe: Fix forceAllocateIfSufficient & Load: Remove RM detect logic during the second phase Mar 13, 2025
@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
2 Security Hotspots

See analysis details on SonarQube Cloud

@SteveYurongSu SteveYurongSu merged commit 0ff85cd into dev/1.3 Mar 14, 2025
24 of 28 checks passed
@SteveYurongSu SteveYurongSu deleted the fix-pipe-stuck-by-file branch March 14, 2025 01:53
SteveYurongSu added a commit that referenced this pull request Mar 14, 2025
…on retries & Pipe: Fix IO triggered in disruptor & Pipe: Fix forceAllocateIfSufficient & Load: Remove RM detect logic during the second phase (#15085)

Co-authored-by: VGalaxies <vgalaxies@apache.org>
SteveYurongSu added a commit that referenced this pull request Mar 14, 2025
…on retries & Pipe: Fix IO triggered in disruptor & Pipe: Fix forceAllocateIfSufficient & Load: Remove RM detect logic during the second phase (#15085) (#15091)

Co-authored-by: VGalaxies <vgalaxies@apache.org>
JackieTien97 pushed a commit that referenced this pull request Apr 14, 2025
…on retries & Pipe: Fix IO triggered in disruptor & Pipe: Fix forceAllocateIfSufficient & Load: Remove RM detect logic during the second phase (#15085) (#15091)

Co-authored-by: VGalaxies <vgalaxies@apache.org>
(cherry picked from commit 69fca07)
Caideyipi pushed a commit to Caideyipi/iotdb that referenced this pull request Mar 25, 2026
…on retries & Pipe: Fix IO triggered in disruptor & Pipe: Fix forceAllocateIfSufficient & Load: Remove RM detect logic during the second phase (apache#15085)

Co-authored-by: VGalaxies <vgalaxies@apache.org>
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.

3 participants