iotdb icon indicating copy to clipboard operation
iotdb copied to clipboard

Opt for only unpacking tsfile when it's really needed

Open JackieTien97 opened this issue 2 years ago • 2 comments

Previously, we always unpack one seq and one unseq tsfile in hasNextFile. image

In this pr, we try to unpack tsfile which we really need instead of unpacking at least one seq and one unseq timeseries metadata each time. In some case that has limit clasue, if the seq and unseq timeseries metadata are not overlapped, we can save one disk IO(if cache missed).

JackieTien97 avatar Dec 06 '23 10:12 JackieTien97

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 3 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

sonarqubecloud[bot] avatar Dec 06 '23 10:12 sonarqubecloud[bot]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (4a9fb92) 49.02% compared to head (6e1650f) 49.05%. Report is 5 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master   #11669      +/-   ##
============================================
+ Coverage     49.02%   49.05%   +0.02%     
- Complexity    24936    24958      +22     
============================================
  Files          2814     2814              
  Lines        176434   176497      +63     
  Branches      21185    21205      +20     
============================================
+ Hits          86500    86581      +81     
+ Misses        89934    89916      -18     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov-commenter avatar Dec 06 '23 10:12 codecov-commenter