search

owncloud / data_exporter

Export/Import for ownCloud user data
GNU General Public License v2.0
7 stars 5 forks source link

Don't copy whole file in to memory during export #120

Closed IljaN closed 5 years ago

IljaN commented 5 years ago

The file was read in to memory during export. This lead to OOM with bigger files (e.g vm Images)

Close #28

codecov[bot] commented 5 years ago

Codecov Report

Merging #120 into master will increase coverage by 0.03%. The diff coverage is 90%.

Impacted file tree graph

@@             Coverage Diff             @@
##             master    #120      +/-   ##
===========================================
+ Coverage     86.77%   86.8%   +0.03%     
- Complexity      186     188       +2     
===========================================
  Files            31      31              
  Lines           892     902      +10     
===========================================
+ Hits            774     783       +9     
- Misses          118     119       +1
Impacted Files Coverage Δ Complexity Δ
lib/Extractor/FilesExtractor.php 100% <ø> (ø) 3 <0> (ø) :arrow_down:
lib/Importer/FilesImporter.php 98% <90%> (-2%) 9 <0> (+2)

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 5b44485...31fa8d9. Read the comment docs.