Always use Unsafe as the default PinotBufferFactory (apache#13639)

rajagopr · Jul 29, 2024 · 0d33790 · 0d33790
1 parent a6cf197
commit 0d33790
Showing 1 changed file with 2 additions and 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
@@ -167,13 +167,8 @@ public static PinotBufferFactory createDefaultFactory(boolean prioritizeByteBuff
     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);
   }