Improve FileState.ofFile() overheads

We do not need the actual bytes read, hence we loop using a small-ish
buffer.

JIRA: YANGTOOLS-745
Change-Id: I2e35e3965c0a30a3d8368eeee45f5485a1765d29
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

diff --git a/plugin/yang-maven-plugin/src/main/java/org/opendaylight/yangtools/yang2sources/plugin/FileState.java b/plugin/yang-maven-plugin/src/main/java/org/opendaylight/yangtools/yang2sources/plugin/FileState.java
index f3601a5b002d89e5963c6700a3dbb1e0562a16f6..dea2822197c02c5c10f2c07c6f05da58516ee803 100644 (file)
--- a/plugin/yang-maven-plugin/src/main/java/org/opendaylight/yangtools/yang2sources/plugin/FileState.java
+++ b/plugin/yang-maven-plugin/src/main/java/org/opendaylight/yangtools/yang2sources/plugin/FileState.java
@@ -35,6 +35,8 @@ record FileState(@NonNull String path, long size, int crc32) {
         void writeTo(@NonNull OutputStream out) throws IOException;
     }
 
+    private static final int READ_BUFFER_SIZE = 8192;
+
     FileState {
         requireNonNull(path);
     }
@@ -45,7 +47,11 @@ record FileState(@NonNull String path, long size, int crc32) {
 
     static @NonNull FileState ofFile(final Path file) throws IOException {
         try (var cis = new CapturingInputStream(Files.newInputStream(file))) {
-            cis.readAllBytes();
+            // Essentially cis.readAllBytes() except we do not need the actual bytes
+            final var bytes = new byte[READ_BUFFER_SIZE];
+            while (cis.readNBytes(bytes, 0, READ_BUFFER_SIZE) != 0) {
+                // Nothing else
+            }
             return new FileState(file.toString(), cis.size(), cis.crc32c());
         }
     }