Merge pull request #78 from seanf/JENKINS-26254-use-buffer

JENKINS-26254 Use BufferedInputStream to read .exec files
jenkinsci · Dec 3, 2016 · a35ce73 · a35ce73
2 parents c6bf58a + ff66560
commit a35ce73
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/src/main/java/hudson/plugins/jacoco/ExecutionFileLoader.java b/src/main/java/hudson/plugins/jacoco/ExecutionFileLoader.java
@@ -2,9 +2,11 @@
 
 import hudson.FilePath;
 
+import java.io.BufferedInputStream;
 import java.io.File;
 import java.io.FileInputStream;
 import java.io.IOException;
+import java.io.InputStream;
 import java.io.Serializable;
 import java.util.ArrayList;
 import java.util.Arrays;
@@ -84,14 +86,15 @@ private void loadExecutionData() throws IOException {
 			for (FilePath filePath : execFiles) {
 				File executionDataFile = new File(filePath.getRemote());
 				try {
-					final FileInputStream fis = new FileInputStream(executionDataFile);
+					final InputStream inputStream = new BufferedInputStream(
+							new FileInputStream(executionDataFile));
 					try {
-	                    final ExecutionDataReader reader = new ExecutionDataReader(fis);
+	                    final ExecutionDataReader reader = new ExecutionDataReader(inputStream);
 	                    reader.setSessionInfoVisitor(sessionInfoStore);
 	                    reader.setExecutionDataVisitor(executionDataStore);
 	                    reader.read();
 					} finally {
-					    fis.close();
+					    inputStream.close();
 					}
 	            } catch (final IOException e) {
 	            	System.out.println("While reading execution data-file: " + executionDataFile);