[PDFBOX-3730] Encapsulate outputStream into BufferedOutputStream in PDDocument.save()

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.0.5
• Fix Version/s: 2.0.6, 3.0.0 PDFBox
• Component/s: Documentation, Writing
• Assignee: Tilman Hausherr
• Reporter: Peter Pinnau
• Created: 22/Mar/17 16:05
• Updated: 15/May/17 20:28
• Resolved: 23/Mar/17 19:50

Description

I recognized very slow performance of PDDocument.save() with a FileOutputStream over a VPN connection.
I did some testing and found out that using PDDocument.save(File file) solved the issue. So the BufferedInputStream makes the difference.

Since my document API uses streams I encapsulate the OutputStream in a BufferedOutputStream if neccessary:

if (!(outputStream instanceof BufferedOutputStream)) {
outputStream = new BufferedOutputStream(outputStream);
}

pdDocument.save(outputStream);

PDDocument.load() already does that with the InputStream. So I wonder if that would be an improvement for save(), too?

Issue Links

• relates to: PDFBOX-3121 Buffer save(File)