Skip to content
New issue

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

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

Already on GitHub? # to your account

Pdf rendering: try to reduce memory consumption: don't load the fonts in memory, use temp files #481

Closed
syjer opened this issue Jun 25, 2018 · 2 comments
Assignees

Comments

@syjer
Copy link
Member

syjer commented Jun 25, 2018

see d814a3a .

Note, we must wait the next release of openhtmltopdf so that danfickle/openhtmltopdf#215 is included.

@syjer syjer self-assigned this Jun 25, 2018
@syjer
Copy link
Member Author

syjer commented Jun 26, 2018

actually: I've committed a fix to only load on demand: df7179d .

But still, we need to measure the effective memory consumption of the font embedding process/pdf generation and find a way to reduce it: we will need to check pdfbox and openhtmltopdf.

I'll leave open this task.

Edit: this commit does not work when launched from the executable war... Will need a new release of openhtmltopdf.

@syjer syjer changed the title Pdf rendering: don't load the files in memory, use temp files Pdf rendering: try to reduce memory consumption: don't load the files in memory, use temp files Jun 26, 2018
@syjer syjer changed the title Pdf rendering: try to reduce memory consumption: don't load the files in memory, use temp files Pdf rendering: try to reduce memory consumption: don't load the fonts in memory, use temp files Jun 26, 2018
@cbellone
Copy link
Member

can be considered done

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

No branches or pull requests

2 participants