Skip to content

Conversation

@jt2594838
Copy link
Contributor

To increase encoding efficiency, when encoding nulls in a Tablet, the previous value will be used as the placeholder for nulls.
When the previous value itself is null, an exception will be raised.

Instead of using the previous value, the last non-null value (or a default value if none exists) is used as the placeholder.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Aug 5, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
9.2% Duplication on New Code (required ≤ 5%)

See analysis details on SonarQube Cloud

@codecov
Copy link

codecov bot commented Aug 5, 2025

Codecov Report

❌ Patch coverage is 27.27273% with 16 lines in your changes missing coverage. Please review.
✅ Project coverage is 38.95%. Comparing base (7384701) to head (893cdd0).
⚠️ Report is 10 commits behind head on master.

Files with missing lines Patch % Lines
...va/org/apache/iotdb/session/util/SessionUtils.java 27.27% 16 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##             master   #16107   +/-   ##
=========================================
  Coverage     38.94%   38.95%           
  Complexity      198      198           
=========================================
  Files          4878     4876    -2     
  Lines        318186   318187    +1     
  Branches      40126    40128    +2     
=========================================
+ Hits         123923   123939   +16     
+ Misses       194263   194248   -15     

☔ 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 854f1dc into master Aug 6, 2025
29 of 30 checks passed
@jt2594838 jt2594838 deleted the fix_continous_null_encoding branch August 6, 2025 03:26
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