[ 
https://issues.apache.org/jira/browse/PARQUET-2375?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17785759#comment-17785759
 ] 

ASF GitHub Bot commented on PARQUET-2375:
-----------------------------------------

jatin-bhateja commented on code in PR #1186:
URL: https://github.com/apache/parquet-mr/pull/1186#discussion_r1392041534


##########
parquet-plugins/parquet-plugins-benchmarks/src/main/java/org/apache/parquet/plugins/benchmarks/ByteBitPackingVectorBenchmarks.java:
##########
@@ -38,48 +40,60 @@
  */
 
 @State(Scope.Benchmark)
-@BenchmarkMode(Mode.AverageTime)
+@BenchmarkMode(Mode.Throughput)
 @Warmup(iterations = 1, batchSize = 100000)
 @Measurement(iterations = 1, batchSize = 100000)
-@OutputTimeUnit(TimeUnit.MILLISECONDS)
+@OutputTimeUnit(TimeUnit.SECONDS)
+@Fork(jvmArgsPrepend = {"--add-modules=jdk.incubator.vector"})

Review Comment:
   A transition of VectorAPI from an incubator module will request changes in 
both tests and vectorized packers 
([PARQUET-2159](https://issues.apache.org/jira/browse/PARQUET-2159)) in future.





> Extend vectorized bit unpacking benchmark for various bit sizes.
> ----------------------------------------------------------------
>
>                 Key: PARQUET-2375
>                 URL: https://issues.apache.org/jira/browse/PARQUET-2375
>             Project: Parquet
>          Issue Type: Improvement
>          Components: parquet-mr
>            Reporter: JATIN BHATEJA
>            Priority: Minor
>
> * Cleanup the benchmark and remove noise from main benchmarking kernels.
>  * Extend the benchmark for various bit widths (1-31)



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to