This is an automated email from the ASF dual-hosted git repository.

lidavidm pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow-java.git


The following commit(s) were added to refs/heads/main by this push:
     new 3dd03115 GH-41: Add VariableWidthViewVectorBenchmarks (#453)
3dd03115 is described below

commit 3dd0311592f7ca8db615791b4f8fd3aa8dea71cb
Author: ViggoC <[email protected]>
AuthorDate: Fri Jan 24 12:08:43 2025 +0800

    GH-41: Add VariableWidthViewVectorBenchmarks (#453)
    
    GitHub Issue: https://github.com/apache/arrow-java/issues/41
---
 .../VariableWidthVectorBenchmarkConstants.java     | 22 +++++++++++++++++
 .../vector/VariableWidthVectorBenchmarks.java      | 13 ++++++----
 ... VariableWidthVectorInlineValueBenchmarks.java} | 17 ++++++++-----
 ...java => VariableWidthViewVectorBenchmarks.java} | 27 ++++++++++++---------
 ...iableWidthViewVectorInlineValueBenchmarks.java} | 28 +++++++++++++---------
 5 files changed, 75 insertions(+), 32 deletions(-)

diff --git 
a/performance/src/main/java/org/apache/arrow/vector/VariableWidthVectorBenchmarkConstants.java
 
b/performance/src/main/java/org/apache/arrow/vector/VariableWidthVectorBenchmarkConstants.java
new file mode 100644
index 00000000..d4ef49d9
--- /dev/null
+++ 
b/performance/src/main/java/org/apache/arrow/vector/VariableWidthVectorBenchmarkConstants.java
@@ -0,0 +1,22 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *    http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.arrow.vector;
+
+public class VariableWidthVectorBenchmarkConstants {
+  public static final String SHORT_VALUE = "InlineValue";
+  public static final String LONG_VALUE = 
VariableWidthVectorBenchmarks.class.getName();
+}
diff --git 
a/performance/src/main/java/org/apache/arrow/vector/VariableWidthVectorBenchmarks.java
 
b/performance/src/main/java/org/apache/arrow/vector/VariableWidthVectorBenchmarks.java
index 0bce6569..17d3ac1f 100644
--- 
a/performance/src/main/java/org/apache/arrow/vector/VariableWidthVectorBenchmarks.java
+++ 
b/performance/src/main/java/org/apache/arrow/vector/VariableWidthVectorBenchmarks.java
@@ -23,8 +23,10 @@ import org.apache.arrow.memory.RootAllocator;
 import org.apache.arrow.vector.holders.NullableVarCharHolder;
 import org.openjdk.jmh.annotations.Benchmark;
 import org.openjdk.jmh.annotations.BenchmarkMode;
+import org.openjdk.jmh.annotations.Level;
 import org.openjdk.jmh.annotations.Mode;
 import org.openjdk.jmh.annotations.OutputTimeUnit;
+import org.openjdk.jmh.annotations.Param;
 import org.openjdk.jmh.annotations.Scope;
 import org.openjdk.jmh.annotations.Setup;
 import org.openjdk.jmh.annotations.State;
@@ -45,15 +47,18 @@ public class VariableWidthVectorBenchmarks {
 
   private static final int ALLOCATOR_CAPACITY = 1024 * 1024;
 
-  private static byte[] bytes = 
VariableWidthVectorBenchmarks.class.getName().getBytes();
+  private static byte[] bytes = 
VariableWidthVectorBenchmarkConstants.LONG_VALUE.getBytes();
   private ArrowBuf arrowBuff;
 
   private BufferAllocator allocator;
 
   private VarCharVector vector;
 
+  @Param({"1", "2", "10", "40"})
+  private int step;
+
   /** Setup benchmarks. */
-  @Setup
+  @Setup(Level.Iteration)
   public void prepare() {
     allocator = new RootAllocator(ALLOCATOR_CAPACITY);
     vector = new VarCharVector("vector", allocator);
@@ -63,7 +68,7 @@ public class VariableWidthVectorBenchmarks {
   }
 
   /** Tear down benchmarks. */
-  @TearDown
+  @TearDown(Level.Iteration)
   public void tearDown() {
     arrowBuff.close();
     vector.close();
@@ -87,7 +92,7 @@ public class VariableWidthVectorBenchmarks {
   @OutputTimeUnit(TimeUnit.MILLISECONDS)
   public int setSafeFromArray() {
     for (int i = 0; i < 500; ++i) {
-      vector.setSafe(i * 40, bytes);
+      vector.setSafe(i * step, bytes);
     }
     return vector.getBufferSize();
   }
diff --git 
a/performance/src/main/java/org/apache/arrow/vector/VariableWidthVectorBenchmarks.java
 
b/performance/src/main/java/org/apache/arrow/vector/VariableWidthVectorInlineValueBenchmarks.java
similarity index 88%
copy from 
performance/src/main/java/org/apache/arrow/vector/VariableWidthVectorBenchmarks.java
copy to 
performance/src/main/java/org/apache/arrow/vector/VariableWidthVectorInlineValueBenchmarks.java
index 0bce6569..354d1b14 100644
--- 
a/performance/src/main/java/org/apache/arrow/vector/VariableWidthVectorBenchmarks.java
+++ 
b/performance/src/main/java/org/apache/arrow/vector/VariableWidthVectorInlineValueBenchmarks.java
@@ -23,8 +23,10 @@ import org.apache.arrow.memory.RootAllocator;
 import org.apache.arrow.vector.holders.NullableVarCharHolder;
 import org.openjdk.jmh.annotations.Benchmark;
 import org.openjdk.jmh.annotations.BenchmarkMode;
+import org.openjdk.jmh.annotations.Level;
 import org.openjdk.jmh.annotations.Mode;
 import org.openjdk.jmh.annotations.OutputTimeUnit;
+import org.openjdk.jmh.annotations.Param;
 import org.openjdk.jmh.annotations.Scope;
 import org.openjdk.jmh.annotations.Setup;
 import org.openjdk.jmh.annotations.State;
@@ -36,7 +38,7 @@ import org.openjdk.jmh.runner.options.OptionsBuilder;
 
 /** Benchmarks for {@link BaseVariableWidthVector}. */
 @State(Scope.Benchmark)
-public class VariableWidthVectorBenchmarks {
+public class VariableWidthVectorInlineValueBenchmarks {
   // checkstyle:off: MissingJavadocMethod
 
   private static final int VECTOR_CAPACITY = 16 * 1024;
@@ -45,15 +47,18 @@ public class VariableWidthVectorBenchmarks {
 
   private static final int ALLOCATOR_CAPACITY = 1024 * 1024;
 
-  private static byte[] bytes = 
VariableWidthVectorBenchmarks.class.getName().getBytes();
+  private static final byte[] bytes = 
VariableWidthVectorBenchmarkConstants.SHORT_VALUE.getBytes();
   private ArrowBuf arrowBuff;
 
   private BufferAllocator allocator;
 
   private VarCharVector vector;
 
+  @Param({"1", "2", "10", "40"})
+  private int step;
+
   /** Setup benchmarks. */
-  @Setup
+  @Setup(Level.Iteration)
   public void prepare() {
     allocator = new RootAllocator(ALLOCATOR_CAPACITY);
     vector = new VarCharVector("vector", allocator);
@@ -63,7 +68,7 @@ public class VariableWidthVectorBenchmarks {
   }
 
   /** Tear down benchmarks. */
-  @TearDown
+  @TearDown(Level.Iteration)
   public void tearDown() {
     arrowBuff.close();
     vector.close();
@@ -87,7 +92,7 @@ public class VariableWidthVectorBenchmarks {
   @OutputTimeUnit(TimeUnit.MILLISECONDS)
   public int setSafeFromArray() {
     for (int i = 0; i < 500; ++i) {
-      vector.setSafe(i * 40, bytes);
+      vector.setSafe(i * step, bytes);
     }
     return vector.getBufferSize();
   }
@@ -115,7 +120,7 @@ public class VariableWidthVectorBenchmarks {
   public static void main(String[] args) throws RunnerException {
     Options opt =
         new OptionsBuilder()
-            .include(VariableWidthVectorBenchmarks.class.getSimpleName())
+            
.include(VariableWidthVectorInlineValueBenchmarks.class.getSimpleName())
             .forks(1)
             .build();
 
diff --git 
a/performance/src/main/java/org/apache/arrow/vector/VariableWidthVectorBenchmarks.java
 
b/performance/src/main/java/org/apache/arrow/vector/VariableWidthViewVectorBenchmarks.java
similarity index 82%
copy from 
performance/src/main/java/org/apache/arrow/vector/VariableWidthVectorBenchmarks.java
copy to 
performance/src/main/java/org/apache/arrow/vector/VariableWidthViewVectorBenchmarks.java
index 0bce6569..b85377dc 100644
--- 
a/performance/src/main/java/org/apache/arrow/vector/VariableWidthVectorBenchmarks.java
+++ 
b/performance/src/main/java/org/apache/arrow/vector/VariableWidthViewVectorBenchmarks.java
@@ -20,11 +20,13 @@ import java.util.concurrent.TimeUnit;
 import org.apache.arrow.memory.ArrowBuf;
 import org.apache.arrow.memory.BufferAllocator;
 import org.apache.arrow.memory.RootAllocator;
-import org.apache.arrow.vector.holders.NullableVarCharHolder;
+import org.apache.arrow.vector.holders.NullableViewVarCharHolder;
 import org.openjdk.jmh.annotations.Benchmark;
 import org.openjdk.jmh.annotations.BenchmarkMode;
+import org.openjdk.jmh.annotations.Level;
 import org.openjdk.jmh.annotations.Mode;
 import org.openjdk.jmh.annotations.OutputTimeUnit;
+import org.openjdk.jmh.annotations.Param;
 import org.openjdk.jmh.annotations.Scope;
 import org.openjdk.jmh.annotations.Setup;
 import org.openjdk.jmh.annotations.State;
@@ -36,7 +38,7 @@ import org.openjdk.jmh.runner.options.OptionsBuilder;
 
 /** Benchmarks for {@link BaseVariableWidthVector}. */
 @State(Scope.Benchmark)
-public class VariableWidthVectorBenchmarks {
+public class VariableWidthViewVectorBenchmarks {
   // checkstyle:off: MissingJavadocMethod
 
   private static final int VECTOR_CAPACITY = 16 * 1024;
@@ -45,25 +47,28 @@ public class VariableWidthVectorBenchmarks {
 
   private static final int ALLOCATOR_CAPACITY = 1024 * 1024;
 
-  private static byte[] bytes = 
VariableWidthVectorBenchmarks.class.getName().getBytes();
+  private static byte[] bytes = 
VariableWidthVectorBenchmarkConstants.LONG_VALUE.getBytes();
   private ArrowBuf arrowBuff;
 
   private BufferAllocator allocator;
 
-  private VarCharVector vector;
+  private ViewVarCharVector vector;
+
+  @Param({"1", "2", "10", "40"})
+  private int step;
 
   /** Setup benchmarks. */
-  @Setup
+  @Setup(Level.Iteration)
   public void prepare() {
-    allocator = new RootAllocator(ALLOCATOR_CAPACITY);
-    vector = new VarCharVector("vector", allocator);
+    allocator = new RootAllocator();
+    vector = new ViewVarCharVector("vector", allocator);
     vector.allocateNew(VECTOR_CAPACITY, VECTOR_LENGTH);
     arrowBuff = allocator.buffer(VECTOR_LENGTH);
     arrowBuff.setBytes(0, bytes, 0, bytes.length);
   }
 
   /** Tear down benchmarks. */
-  @TearDown
+  @TearDown(Level.Iteration)
   public void tearDown() {
     arrowBuff.close();
     vector.close();
@@ -87,7 +92,7 @@ public class VariableWidthVectorBenchmarks {
   @OutputTimeUnit(TimeUnit.MILLISECONDS)
   public int setSafeFromArray() {
     for (int i = 0; i < 500; ++i) {
-      vector.setSafe(i * 40, bytes);
+      vector.setSafe(i * step, bytes);
     }
     return vector.getBufferSize();
   }
@@ -96,7 +101,7 @@ public class VariableWidthVectorBenchmarks {
   @BenchmarkMode(Mode.AverageTime)
   @OutputTimeUnit(TimeUnit.MILLISECONDS)
   public int setSafeFromNullableVarcharHolder() {
-    NullableVarCharHolder nvch = new NullableVarCharHolder();
+    NullableViewVarCharHolder nvch = new NullableViewVarCharHolder();
     nvch.buffer = arrowBuff;
     nvch.start = 0;
     nvch.end = bytes.length;
@@ -115,7 +120,7 @@ public class VariableWidthVectorBenchmarks {
   public static void main(String[] args) throws RunnerException {
     Options opt =
         new OptionsBuilder()
-            .include(VariableWidthVectorBenchmarks.class.getSimpleName())
+            .include(VariableWidthViewVectorBenchmarks.class.getSimpleName())
             .forks(1)
             .build();
 
diff --git 
a/performance/src/main/java/org/apache/arrow/vector/VariableWidthVectorBenchmarks.java
 
b/performance/src/main/java/org/apache/arrow/vector/VariableWidthViewVectorInlineValueBenchmarks.java
similarity index 81%
copy from 
performance/src/main/java/org/apache/arrow/vector/VariableWidthVectorBenchmarks.java
copy to 
performance/src/main/java/org/apache/arrow/vector/VariableWidthViewVectorInlineValueBenchmarks.java
index 0bce6569..32143d47 100644
--- 
a/performance/src/main/java/org/apache/arrow/vector/VariableWidthVectorBenchmarks.java
+++ 
b/performance/src/main/java/org/apache/arrow/vector/VariableWidthViewVectorInlineValueBenchmarks.java
@@ -20,11 +20,13 @@ import java.util.concurrent.TimeUnit;
 import org.apache.arrow.memory.ArrowBuf;
 import org.apache.arrow.memory.BufferAllocator;
 import org.apache.arrow.memory.RootAllocator;
-import org.apache.arrow.vector.holders.NullableVarCharHolder;
+import org.apache.arrow.vector.holders.NullableViewVarCharHolder;
 import org.openjdk.jmh.annotations.Benchmark;
 import org.openjdk.jmh.annotations.BenchmarkMode;
+import org.openjdk.jmh.annotations.Level;
 import org.openjdk.jmh.annotations.Mode;
 import org.openjdk.jmh.annotations.OutputTimeUnit;
+import org.openjdk.jmh.annotations.Param;
 import org.openjdk.jmh.annotations.Scope;
 import org.openjdk.jmh.annotations.Setup;
 import org.openjdk.jmh.annotations.State;
@@ -36,7 +38,7 @@ import org.openjdk.jmh.runner.options.OptionsBuilder;
 
 /** Benchmarks for {@link BaseVariableWidthVector}. */
 @State(Scope.Benchmark)
-public class VariableWidthVectorBenchmarks {
+public class VariableWidthViewVectorInlineValueBenchmarks {
   // checkstyle:off: MissingJavadocMethod
 
   private static final int VECTOR_CAPACITY = 16 * 1024;
@@ -45,25 +47,29 @@ public class VariableWidthVectorBenchmarks {
 
   private static final int ALLOCATOR_CAPACITY = 1024 * 1024;
 
-  private static byte[] bytes = 
VariableWidthVectorBenchmarks.class.getName().getBytes();
+  private static byte[] bytes = 
VariableWidthVectorBenchmarkConstants.SHORT_VALUE.getBytes();
   private ArrowBuf arrowBuff;
 
   private BufferAllocator allocator;
 
-  private VarCharVector vector;
+  private ViewVarCharVector vector;
+
+  @Param({"1", "2", "10", "40"})
+  private int step;
 
   /** Setup benchmarks. */
-  @Setup
+  @Setup(Level.Invocation)
   public void prepare() {
-    allocator = new RootAllocator(ALLOCATOR_CAPACITY);
-    vector = new VarCharVector("vector", allocator);
+    allocator = new RootAllocator();
+    vector = new ViewVarCharVector("vector", allocator);
     vector.allocateNew(VECTOR_CAPACITY, VECTOR_LENGTH);
+    vector.zeroVector();
     arrowBuff = allocator.buffer(VECTOR_LENGTH);
     arrowBuff.setBytes(0, bytes, 0, bytes.length);
   }
 
   /** Tear down benchmarks. */
-  @TearDown
+  @TearDown(Level.Invocation)
   public void tearDown() {
     arrowBuff.close();
     vector.close();
@@ -87,7 +93,7 @@ public class VariableWidthVectorBenchmarks {
   @OutputTimeUnit(TimeUnit.MILLISECONDS)
   public int setSafeFromArray() {
     for (int i = 0; i < 500; ++i) {
-      vector.setSafe(i * 40, bytes);
+      vector.setSafe(i * step, bytes);
     }
     return vector.getBufferSize();
   }
@@ -96,7 +102,7 @@ public class VariableWidthVectorBenchmarks {
   @BenchmarkMode(Mode.AverageTime)
   @OutputTimeUnit(TimeUnit.MILLISECONDS)
   public int setSafeFromNullableVarcharHolder() {
-    NullableVarCharHolder nvch = new NullableVarCharHolder();
+    NullableViewVarCharHolder nvch = new NullableViewVarCharHolder();
     nvch.buffer = arrowBuff;
     nvch.start = 0;
     nvch.end = bytes.length;
@@ -115,7 +121,7 @@ public class VariableWidthVectorBenchmarks {
   public static void main(String[] args) throws RunnerException {
     Options opt =
         new OptionsBuilder()
-            .include(VariableWidthVectorBenchmarks.class.getSimpleName())
+            
.include(VariableWidthViewVectorInlineValueBenchmarks.class.getSimpleName())
             .forks(1)
             .build();
 

Reply via email to