(pinot) branch master updated: Always use Unsafe as the default PinotBufferFactory (#13639)

[gortiz]

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

gortiz pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git


The following commit(s) were added to refs/heads/master by this push:
     new 0d33790c32 Always use Unsafe as the default PinotBufferFactory (#13639)
0d33790c32 is described below

commit 0d33790c32c8d318cbf6cede07073637038b6670
Author: Gonzalo Ortiz Jaureguizar <gor...@users.noreply.github.com>
AuthorDate: Mon Jul 29 07:35:58 2024 +0200

    Always use Unsafe as the default PinotBufferFactory (#13639)
---
 .../org/apache/pinot/segment/spi/memory/PinotDataBuffer.java     | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git 
a/pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/memory/PinotDataBuffer.java
 
b/pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/memory/PinotDataBuffer.java
index 2ed9d22e92..c2fb5084a7 100644
--- 
a/pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/memory/PinotDataBuffer.java
+++ 
b/pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/memory/PinotDataBuffer.java
@@ -167,13 +167,8 @@ public abstract class PinotDataBuffer implements 
DataBuffer {
     String factoryClassName;
     factoryClassName = System.getenv("PINOT_BUFFER_LIBRARY");
     if (factoryClassName == null) {
-      if (JavaVersion.VERSION < 16) {
-        LOGGER.info("Using LArray as buffer on JVM version {}", 
JavaVersion.VERSION);
-        factoryClassName = LArrayPinotBufferFactory.class.getCanonicalName();
-      } else {
-        LOGGER.info("Using Unsafe as buffer on JVM version {}", 
JavaVersion.VERSION);
-        factoryClassName = UnsafePinotBufferFactory.class.getCanonicalName();
-      }
+      LOGGER.info("Using Unsafe as buffer on JVM version {}", 
JavaVersion.VERSION);
+      factoryClassName = UnsafePinotBufferFactory.class.getCanonicalName();
     }
     return createFactory(factoryClassName, prioritizeByteBuffer);
   }


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org