This is an automated email from the ASF dual-hosted git repository. bchapuis pushed a commit to branch workflow in repository https://gitbox.apache.org/repos/asf/incubator-baremaps.git
commit e2554064a18918322e060e9086f6d4a0b050d9c8 Author: Bertil Chapuis <[email protected]> AuthorDate: Fri Nov 17 16:44:34 2023 +0100 Simplify file decompression --- .../org/apache/baremaps/cli/database/UpdateOsm.java | 2 +- .../apache/baremaps/workflow/tasks/DecompressFile.java | 18 +++--------------- 2 files changed, 4 insertions(+), 16 deletions(-) diff --git a/baremaps-cli/src/main/java/org/apache/baremaps/cli/database/UpdateOsm.java b/baremaps-cli/src/main/java/org/apache/baremaps/cli/database/UpdateOsm.java index 832a6f72..11fb8d6c 100644 --- a/baremaps-cli/src/main/java/org/apache/baremaps/cli/database/UpdateOsm.java +++ b/baremaps-cli/src/main/java/org/apache/baremaps/cli/database/UpdateOsm.java @@ -42,7 +42,7 @@ public class UpdateOsm implements Callable<Integer> { private int srid = 3857; @Option(names = {"--replication-url"}, paramLabel = "REPLICATION_URL", - description = "The replication url of the OpenStreetMap server.") + description = "The replication url of the OpenStreetMap server.") private String replicationUrl = "https://planet.osm.org/replication/hour"; @Override diff --git a/baremaps-core/src/main/java/org/apache/baremaps/workflow/tasks/DecompressFile.java b/baremaps-core/src/main/java/org/apache/baremaps/workflow/tasks/DecompressFile.java index 272ad1a3..e5a4108b 100644 --- a/baremaps-core/src/main/java/org/apache/baremaps/workflow/tasks/DecompressFile.java +++ b/baremaps-core/src/main/java/org/apache/baremaps/workflow/tasks/DecompressFile.java @@ -133,11 +133,7 @@ public class DecompressFile implements Task { Files.createDirectories(path.getParent()); try (BufferedOutputStream outputStream = new BufferedOutputStream(Files.newOutputStream(path))) { - int bytesRead; - byte[] buffer = new byte[4096]; - while ((bytesRead = tarInputStream.read(buffer)) != -1) { - outputStream.write(buffer, 0, bytesRead); - } + tarInputStream.transferTo(outputStream); } } } @@ -164,11 +160,7 @@ public class DecompressFile implements Task { Files.createDirectories(path.getParent()); try (BufferedOutputStream outputStream = new BufferedOutputStream(Files.newOutputStream(path))) { - int bytesRead; - byte[] buffer = new byte[4096]; - while ((bytesRead = tarInputStream.read(buffer)) != -1) { - outputStream.write(buffer, 0, bytesRead); - } + tarInputStream.transferTo(outputStream); } } } @@ -193,11 +185,7 @@ public class DecompressFile implements Task { StandardOpenOption.TRUNCATE_EXISTING); try (var input = new BufferedInputStream(zipFile.getInputStream(entry)); var output = new BufferedOutputStream(new FileOutputStream(path.toFile()))) { - int nBytes = -1; - byte[] buffer = new byte[4096]; - while ((nBytes = input.read(buffer)) > 0) { - output.write(buffer, 0, nBytes); - } + input.transferTo(output); } } }
