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

Update PropertyResolver.java #262

Open
wants to merge 3 commits into
base: VERSION_2_8_1
Choose a base branch
from

Conversation

shelatgauri
Copy link

@shelatgauri shelatgauri commented Sep 14, 2017

without_deepcopy

with_deepcopy
For documents which are created using template, the default Rpr can be deep-copied once and saved in threadlocal. I have attached snapshots of with and without changes, to give an idea of amount of time saved.

The getEffectiveRPr does a deep copy. This is done using marshalling unmarshalling . To reduce this time, we can simply reset defaultRpr.
@shelatgauri
Copy link
Author

shelatgauri commented Sep 14, 2017

pdf-1.0-SNAPSHOT.zip
This is a sample test app. To use this war deploy it after making changes(for input and output files) in the in PDFGeneration class and compiling it using the project below:
generate-pdf.zip

@plutext
Copy link
Owner

plutext commented Sep 14, 2017 via email

@shelatgauri
Copy link
Author

Kindly refer to the issue created.
link to the issue : #263.
I am also attaching a sample document containing many run elements.
SampleDoc1.docx

added method to clear the the thread local created. This is done to avoid memory leaks in multi-threaded environment.
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

Successfully merging this pull request may close these issues.

2 participants