Skip to content

[To dev/1.1] Fix VectorMeasurementSchema stream deserialization - #940

Merged
JackieTien97 merged 1 commit into
dev/1.1from
ty/fix-vector-schema-stream-deserialization-1.1
Sep 4, 2026
Merged

[To dev/1.1] Fix VectorMeasurementSchema stream deserialization#940
JackieTien97 merged 1 commit into
dev/1.1from
ty/fix-vector-schema-stream-deserialization-1.1

Conversation

@JackieTien97

Copy link
Copy Markdown
Contributor

Summary

  • fix VectorMeasurementSchema.deserializeFrom(InputStream) to read the complete time/value compressor list
  • preserve unified-compressor behavior and reject truncated per-column compressor data
  • add regression coverage for round trips, short reads, and truncated streams

Compatibility

  • no public API or serialized-format changes
  • valid existing serialized schemas remain readable
  • DataInputStream.readFully is available on Java 8, so the backport introduces no newer runtime requirement
  • streams with truncated compressor data now fail fast instead of being accepted or decoded incorrectly

Testing

  • ./mvnw test -P with-java -pl java/tsfile -am

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 59.09%. Comparing base (b55b270) to head (fc04c3f).

Additional details and impacted files
@@             Coverage Diff             @@
##           dev/1.1     #940      +/-   ##
===========================================
+ Coverage    58.82%   59.09%   +0.26%     
===========================================
  Files          420      420              
  Lines        28117    28118       +1     
  Branches      3718     3716       -2     
===========================================
+ Hits         16541    16615      +74     
+ Misses       11380    11307      -73     
  Partials       196      196              

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

@JackieTien97
JackieTien97 merged commit 396fdf2 into dev/1.1 Sep 4, 2026
10 checks passed
@JackieTien97
JackieTien97 deleted the ty/fix-vector-schema-stream-deserialization-1.1 branch September 4, 2026 07:54
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.

2 participants