[
https://issues.apache.org/jira/browse/PDFBOX-5499?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17600481#comment-17600481
]
Thomas Debray Luyat commented on PDFBOX-5499:
---------------------------------------------
Hi [~tilman] , I'm really sorry it's not really easy to share the PDF file
because it contains customer information. I'll try to edit the document to
obfuscate information.
[~mkl]
"performance issue really is in the parsing and not in the closing" : You're
true, my test measure also document closing.
2.0.26:
!image-2022-09-05-17-37-55-155.png!
2.0.18:
!image-2022-09-05-17-40-22-416.png!
"the {{SmallMap}} is performant only for small numbers of entries" : I'm pretty
sure this is the cause of the issue. The PDF contains kitchen drawing with
multi layers and probably a lot of information.
> Performance issue since 2.0.18
> ------------------------------
>
> Key: PDFBOX-5499
> URL: https://issues.apache.org/jira/browse/PDFBOX-5499
> Project: PDFBox
> Issue Type: Bug
> Components: PDModel
> Affects Versions: 2.0.19
> Reporter: Thomas Debray Luyat
> Priority: Major
> Attachments: image-2022-09-05-12-48-04-608.png,
> image-2022-09-05-17-37-55-155.png, image-2022-09-05-17-40-22-416.png
>
>
> Our PDF is parsed in less than 200ms in 2.0.18 and more then 8 seconds in
> 2.0.19. The same issue is still there in 2.0.26.
>
> In version 2.0.19, SmallMap has been introduced. We're facing a performance
> issue since this modification.
> !image-2022-09-05-12-48-04-608.png|width=968,height=377!
> We patch our code to just replace the SmallMap implementation like this:
> {code:java}
> package org.apache.pdfbox.util;
> import java.util.LinkedHashMap;
> public class SmallMap<K, V> extends LinkedHashMap<K, V> {
> // nothing : use the standard LinkedHashMap
> }{code}
> And the performance issue disappear.
> Our test is really simple:
> {code:java}
> long start = System.currentTimeMillis();
> try (PDDocument document = PDDocument.load(new File(inFile))) {
> // nothing : only parsing is evaluated
> }
> long duration = System.currentTimeMillis() -start;
> assertTrue(duration < 500);{code}
>
> I can understand that the SmallMap can solve issues in some cases, but it is
> possible to implement a factory to create this map and then allow to setup
> which Map implementation we want to use?
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
