Skip to content
This repository has been archived by the owner on Jun 21, 2024. It is now read-only.

Don't read whole file in to memory on importing/exporting #28

Closed
IljaN opened this issue Sep 21, 2018 · 4 comments · Fixed by #120 · May be fixed by #39
Closed

Don't read whole file in to memory on importing/exporting #28

IljaN opened this issue Sep 21, 2018 · 4 comments · Fixed by #120 · May be fixed by #39
Labels
enhancement New feature or request
Milestone
backlog

Comments

@IljaN
Copy link
Contributor

IljaN commented Sep 21, 2018

https://github.com/owncloud/data_exporter/blob/master/lib/Importer/FilesImporter.php#L67
https://github.com/owncloud/data_exporter/blob/master/lib/Exporter/FilesExporter.php#L63
@IljaN IljaN added the enhancement New feature or request label Sep 21, 2018
@IljaN IljaN changed the title Don't read whole while in to memory on importing/exporting Don't read whole file in to memory on importing/exporting Sep 21, 2018
@IljaN IljaN self-assigned this Sep 21, 2018
@IljaN IljaN added this to the development milestone Sep 21, 2018
IljaN added a commit that referenced this issue Sep 24, 2018
@IljaN
Don't read whole file in to memory #28
441f48c
@IljaN IljaN mentioned this issue Sep 24, 2018
Closed
9 tasks
@PVince81
Copy link
Contributor

estimate ?

@IljaN
Copy link
Contributor Author

IljaN commented Sep 25, 2018

Already done, was ~1h, tests will be provided trough #29

@jvillafanez
Copy link
Member

This is done as part of #39. Note that the metadata.json file still needs to be read / written as a whole due to limitations in the serializer.

@jvillafanez jvillafanez mentioned this issue Oct 19, 2018
Open
2 tasks
@PVince81 PVince81 modified the milestones: development, backlog Oct 23, 2018
@micbar
Copy link
Contributor

micbar commented Aug 6, 2019

Fixed by #89

@IljaN IljaN mentioned this issue Oct 24, 2019
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
backlog
4 participants
@IljaN @PVince81 @jvillafanez @micbar