Skip to content

Pipe: Fixed the bug that air gap receiver may not respond in temporary timeout exception & Optimized the directory check in receiver & Fixed the bug that the "skipIfNoPrivileges" may be wrongly reused at receiver & Optimized the configNode pipe logic - #17556

Merged
jt2594838 merged 4 commits into
masterfrom
receiver-fix
Apr 29, 2026

Conversation

@Caideyipi

Copy link
Copy Markdown
Collaborator

Description

As the title said.


This PR has:

  • been self-reviewed.
    • concurrent read
    • concurrent write
    • concurrent read and write
  • added documentation for new or modified features or behaviors.
  • added Javadocs for most classes and all non-trivial methods.
  • added or updated version, license, or notice information
  • added comments explaining the "why" and the intent of the code wherever would not be obvious
    for an unfamiliar reader.
  • added unit tests or modified existing tests to cover new code paths, ensuring the threshold
    for code coverage.
  • added integration tests.
  • been tested in a test IoTDB cluster.

Key changed/added classes (or packages if there are too many classes) in this PR

@CaideyipiCaideyipi changed the title Pipe: Fixed the bug that air gap receiver may not respond in temporary timeout exception & Optimized the directory ix in receiverdPipe: Fixed the bug that air gap receiver may not respond in temporary timeout exception & Optimized the directory check in receiver & Fixed the bug that the "skipIfNoPrivileges" may be wrongly reused at receiverApr 27, 2026
@sonarqubecloud

Copy link
Copy Markdown

@CaideyipiCaideyipi changed the title Pipe: Fixed the bug that air gap receiver may not respond in temporary timeout exception & Optimized the directory check in receiver & Fixed the bug that the "skipIfNoPrivileges" may be wrongly reused at receiverPipe: Fixed the bug that air gap receiver may not respond in temporary timeout exception & Optimized the directory check in receiver & Fixed the bug that the "skipIfNoPrivileges" may be wrongly reused at receiver & Optimized the configNode pipe logicApr 27, 2026
@codecov

codecovBot commented Apr 27, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 47.25275% with 48 lines in your changes missing coverage. Please review.
✅ Project coverage is 39.91%. Comparing base (ce84224) to head (8b1e2ce).
⚠️ Report is 8 commits behind head on master.

Files with missing linesPatch %Lines
...rotocol/iotconsensusv2/IoTConsensusV2Receiver.java0.00%30 Missing ⚠️
...iotdb/commons/pipe/receiver/IoTDBFileReceiver.java54.83%14 Missing ⚠️
...che/iotdb/confignode/manager/ProcedureManager.java0.00%2 Missing ⚠️
...dinator/runtime/heartbeat/PipeHeartbeatParser.java85.71%2 Missing ⚠️
Additional details and impacted files
@@ Coverage Diff @@## master #17556 +/- ##
============================================
+ Coverage 39.85% 39.91% +0.05% 
Complexity 2547 2547 ============================================
Files 5171 5172 +1 Lines 348174 348226 +52 Branches 44489 44501 +12 ============================================
+ Hits 138767 138979 +212 + Misses 209407 209247 -160 

☔ 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 73cf067 into masterApr 29, 2026
30 of 31 checks passed
@jt2594838
jt2594838 deleted the receiver-fix branch April 29, 2026 03:21
jt2594838 pushed a commit that referenced this pull request Apr 30, 2026
…d in temporary timeout exception & Optimized the directory check in receiver & Optimized the configNode pipe logic (#17556) & Optimized the clear logic of Schema Region (#17553) (#17576)
* Optimized the clear logic of Schema Region && Pipe: Fixed the bug that the historical pipe does not work for deletion-only sync (#17553)
* Pipe: Fixed the bug that air gap receiver may not respond in temporary timeout exception & Optimized the directory check in receiver & Optimized the configNode pipe logic (#17556)
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