Skip to content

[dev/1.3] Pipe: Fixed the bug that drop pipe may stuck when disruptor is interrupted or ring buffer full - #17818

Merged
jt2594838 merged 1 commit into
dev/1.3from
backport/17673-dev-1.3
Jun 3, 2026
Merged

[dev/1.3] Pipe: Fixed the bug that drop pipe may stuck when disruptor is interrupted or ring buffer full#17818
jt2594838 merged 1 commit into
dev/1.3from
backport/17673-dev-1.3

Conversation

@Caideyipi

@CaideyipiCaideyipi commented Jun 2, 2026

Copy link
Copy Markdown
Collaborator

Backport #17673 to dev/1.3. Cherry-picked from b614d6d. Verification: .\mvnw.cmd -pl iotdb-core/datanode spotless:apply passed; .\mvnw.cmd -pl iotdb-core/datanode -Dtest=DisruptorShutdownTest test failed during datanode main compilation in unrelated pipe sink/commons API mismatch files before tests ran.

…upted or ring buffer full (#17673)
* Fix interrupt
* spotless
(cherry picked from commit b614d6d)
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate FailedQuality Gate failed

Failed conditions
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@codecov

codecovBot commented Jun 2, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 76.00000% with 12 lines in your changes missing coverage. Please review.
✅ Project coverage is 42.39%. Comparing base (3c959c3) to head (00463d6).

Files with missing linesPatch %Lines
...gion/realtime/assigner/PipeDataRegionAssigner.java66.66%8 Missing ⚠️
...e/dataregion/realtime/assigner/DisruptorQueue.java80.00%2 Missing ⚠️
...ion/realtime/disruptor/MultiProducerSequencer.java66.66%2 Missing ⚠️
Additional details and impacted files
@@ Coverage Diff @@## dev/1.3 #17818 +/- ##
=============================================
+ Coverage 42.38% 42.39% +0.01% 
Complexity 241 241 =============================================
Files 3617 3617 Lines 239532 239570 +38 Branches 29131 29139 +8 =============================================
+ Hits 101517 101563 +46 + Misses 138015 138007 -8 

☔ 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.

@jt2594838
jt2594838 merged commit 7a25873 into dev/1.3Jun 3, 2026
18 of 20 checks passed
@jt2594838
jt2594838 deleted the backport/17673-dev-1.3 branch June 3, 2026 02:43
Sign up for freeto 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

@Caideyipi@jt2594838