Skip to content

Conversation

@jt2594838
Copy link
Contributor

Previously, when there was a type conflict, only the currently written partition would be flushed.
If the write involved more than one partition, the other partition would still meet a type conflict, which would fail the write.

To resolve this, all partitions will be flushed once a type conflict is encountered.

@sonarqubecloud
Copy link

@jt2594838 jt2594838 changed the title Fix cross partition write arfter alter data type Fix cross partition write after alter data type Jan 26, 2026
@codecov
Copy link

codecov bot commented Jan 26, 2026

Codecov Report

❌ Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 39.19%. Comparing base (b0495da) to head (d7f0f11).
⚠️ Report is 4 commits behind head on master.

Files with missing lines Patch % Lines
.../iotdb/db/storageengine/dataregion/DataRegion.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master   #17082      +/-   ##
============================================
+ Coverage     39.18%   39.19%   +0.01%     
  Complexity      282      282              
============================================
  Files          5126     5126              
  Lines        343000   342996       -4     
  Branches      43746    43744       -2     
============================================
+ Hits         134401   134434      +33     
+ Misses       208599   208562      -37     

☔ 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 8e5fcba into master Jan 27, 2026
31 checks passed
@jt2594838 jt2594838 deleted the fix_cross_partition_write_arfter_alter_data_type branch January 27, 2026 01:30
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