Skip to content

[To dev/1.3] 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) & Optimized the clear logic of Schema Region (#17553) - #17576

Merged
jt2594838 merged 2 commits into
dev/1.3from
13-cnbug
Apr 30, 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

…t the historical pipe does not work for deletion-only sync (#17553)
…y timeout exception & Optimized the directory check in receiver & Optimized the configNode pipe logic (#17556)
@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 & Optimized the configNode pipe logic (#17556) & Optimized the clear logic of Schema Region (#17553)[To dev/1.3] 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) & Optimized the clear logic of Schema Region (#17553)Apr 29, 2026
@sonarqubecloud

Copy link
Copy Markdown

@codecov

codecovBot commented Apr 29, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 70.96774% with 18 lines in your changes missing coverage. Please review.
✅ Project coverage is 41.65%. Comparing base (7363d35) to head (992acf7).
⚠️ Report is 5 commits behind head on dev/1.3.

Files with missing linesPatch %Lines
...iotdb/commons/pipe/receiver/IoTDBFileReceiver.java56.25%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 @@## dev/1.3 #17576 +/- ##
=============================================
+ Coverage 41.59% 41.65% +0.05% 
Complexity 234 234 =============================================
Files 3597 3598 +1 Lines 237247 237290 +43 Branches 28727 28737 +10 =============================================
+ Hits 98679 98835 +156 + Misses 138568 138455 -113 

☔ 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 291f01c into dev/1.3Apr 30, 2026
24 of 26 checks passed
@jt2594838
jt2594838 deleted the 13-cnbug branch April 30, 2026 01:38
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