Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove file embedding from PLN XML #42

Open
asmecher opened this issue Sep 17, 2019 · 2 comments
Open

Remove file embedding from PLN XML #42

asmecher opened this issue Sep 17, 2019 · 2 comments

Comments

@asmecher
Copy link
Contributor

Currently there is a heavy memory requirement for PLN deposits, as all files in the XML import/export are embedded into the XML directly, requiring all content to be loaded in memory.

It would be better to have the import/export XML refer to the files by relative path, and store the files inside the bag-it zip file.

@ubermichael
Copy link

I think this is a fine solution. I'd also like to add something (maybe a version element or attribute) to the deposit XML generated by DepositPackage::generateAtomDocument() that differentiates these deposits from the embedded XML.

@asmecher
Copy link
Contributor Author

@defstat, I wonder if this one is worth including in 3.2? It'll mean some work to the native import/export plugin, so can't just be done with a PLN plugin improvement.

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

No branches or pull requests

2 participants