srotya / sidewinder

Fast and scalable timeseries database
http://sidewinder.srotya.com
Apache License 2.0
25 stars 10 forks source link

Improving dod compression algorithm #94

Closed ambud closed 7 years ago

ambud commented 7 years ago

Adding zip compression poc implementation

Changing from exception to null

codecov-io commented 7 years ago

Codecov Report

Merging #94 into development will decrease coverage by 1.67%. The diff coverage is 18.58%.

Impacted file tree graph

@@                Coverage Diff                @@
##             development      #94      +/-   ##
=================================================
- Coverage          51.63%   49.95%   -1.68%     
- Complexity           730      734       +4     
=================================================
  Files                126      129       +3     
  Lines               4551     4750     +199     
  Branches             453      478      +25     
=================================================
+ Hits                2350     2373      +23     
- Misses              2065     2234     +169     
- Partials             136      143       +7
Impacted Files Coverage Δ Complexity Δ
...ya/sidewinder/core/storage/compression/Writer.java 100% <ø> (ø) 1 <0> (ø) :arrow_down:
...winder/core/storage/compression/zip/ZipWriter.java 0% <0%> (ø) 0 <0> (?)
...winder/core/storage/compression/dod/DoDReader.java 0% <0%> (ø) 0 <0> (ø) :arrow_down:
.../srotya/sidewinder/core/storage/StorageEngine.java 52.67% <0%> (-10.09%) 27 <0> (ø)
...com/srotya/sidewinder/core/storage/TimeSeries.java 80.28% <0%> (-4.8%) 47 <0> (ø)
...winder/core/storage/compression/zip/ZipReader.java 0% <0%> (ø) 0 <0> (?)
...storage/compression/byzantine/ByzantineWriter.java 88.59% <100%> (ø) 41 <0> (ø) :arrow_down:
...ya/sidewinder/core/storage/compression/Reader.java 100% <100%> (ø) 1 <1> (?)
...storage/compression/byzantine/ByzantineReader.java 96.72% <100%> (-0.06%) 26 <0> (-1)
...winder/core/storage/compression/dod/DodWriter.java 46.84% <68.51%> (+9.16%) 12 <8> (+7) :arrow_up:
... and 5 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 24093ab...55bc5f9. Read the comment docs.