Skip to content

Pipe: Remove Ratis data region source restriction#18089

Open
Caideyipi wants to merge 1 commit into
masterfrom
fix/remove-pipe-ratis-consensus-restriction
Open

Pipe: Remove Ratis data region source restriction#18089
Caideyipi wants to merge 1 commit into
masterfrom
fix/remove-pipe-ratis-consensus-restriction

Conversation

@Caideyipi

Copy link
Copy Markdown
Collaborator

Summary

  • Remove the data region source validation that rejects pipe insertion extraction when data regions use Ratis consensus.
  • Drop the now-unused i18n message.

Tests

  • mvn spotless:apply -pl iotdb-core/datanode
  • mvn compile -pl iotdb-core/datanode (fails: existing generated thrift/antlr/API mismatch, e.g. missing TShowRepairDataPartitionTableProgressResp and parser contexts)

@sonarqubecloud

sonarqubecloud Bot commented Jul 2, 2026

Copy link
Copy Markdown

@codecov

codecov Bot commented Jul 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 41.52%. Comparing base (220e7a3) to head (30d9096).

Additional details and impacted files
@@             Coverage Diff              @@
##             master   #18089      +/-   ##
============================================
- Coverage     41.65%   41.52%   -0.14%     
  Complexity      318      318              
============================================
  Files          5296     5296              
  Lines        371663   371657       -6     
  Branches      48088    48086       -2     
============================================
- Hits         154819   154326     -493     
- Misses       216844   217331     +487     

☔ View full report in Codecov by Harness.
📢 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.

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.

1 participant