Suppress warnings in benchmarks

apache · Oct 13, 2024 · 5ea6c90 · 5ea6c90
1 parent 1f9cc17
commit 5ea6c90
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 9 deletions.
diff --git a/.gitignore b/.gitignore
@@ -50,7 +50,6 @@ examples/openstreetmap/tiles/
 examples/transformation/*.pbf
 
 # Benchmarking
-baremaps-benchmarking/small/
 baremaps-benchmarking/data/
 
 # Docs

diff --git a/...king/src/main/java/org/apache/baremaps/benchmarking/geoparquet/OvertureMapsBenchmark.java b/...king/src/main/java/org/apache/baremaps/benchmarking/geoparquet/OvertureMapsBenchmark.java
@@ -40,7 +40,7 @@
 @Measurement(iterations = 1)
 public class OvertureMapsBenchmark {
 
-  private static Path directory = Path.of("baremaps-benchmarking/data");
+  private static Path directory = Path.of("baremaps-benchmarking/data/overturemaps");
 
   public static void main(String[] args) throws RunnerException {
     Options opt = new OptionsBuilder()
@@ -80,15 +80,17 @@ public void setup() throws IOException {
     }
   }
 
+  @SuppressWarnings("squid:S1481")
   @Benchmark
   public void read() {
     GeoParquetReader reader = new GeoParquetReader(directory.toUri());
-    long count = reader.read().count();
+    reader.read().count();
   }
 
+  @SuppressWarnings("squid:S1481")
   @Benchmark
   public void readParallel() {
     GeoParquetReader reader = new GeoParquetReader(directory.toUri());
-    long count = reader.readParallel().count();
+    reader.readParallel().count();
   }
 }
diff --git a/...marking/src/main/java/org/apache/baremaps/benchmarking/geoparquet/SmallFileBenchmark.java b/...marking/src/main/java/org/apache/baremaps/benchmarking/geoparquet/SmallFileBenchmark.java
@@ -37,7 +37,7 @@
 public class SmallFileBenchmark {
 
   private Path source = Path.of("baremaps-testing/data/samples/example.parquet").toAbsolutePath();
-  private Path directory = Path.of("baremaps-benchmarking/small").toAbsolutePath();
+  private Path directory = Path.of("baremaps-benchmarking/data/small").toAbsolutePath();
 
   public static void main(String[] args) throws RunnerException {
     Options opt = new OptionsBuilder()
@@ -58,17 +58,19 @@ public void setup() throws IOException {
     }
   }
 
+  @SuppressWarnings("squid:S1481")
   @Benchmark
   public void read() {
     GeoParquetReader reader =
-        new GeoParquetReader(Path.of("baremaps-benchmarking/small/*.parquet").toUri());
-    long count = reader.read().count();
+        new GeoParquetReader(Path.of("baremaps-benchmarking/data/small/*.parquet").toUri());
+    reader.read().count();
   }
 
+  @SuppressWarnings("squid:S1481")
   @Benchmark
   public void readParallel() {
     GeoParquetReader reader =
-        new GeoParquetReader(Path.of("baremaps-benchmarking/small/*.parquet").toUri());
-    long count = reader.readParallel().count();
+        new GeoParquetReader(Path.of("baremaps-benchmarking/data/small/*.parquet").toUri());
+    reader.readParallel().count();
   }
 }