You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by xi...@apache.org on 2019/09/19 09:16:12 UTC

[incubator-pinot] 01/01: Remove import of sun.misc.Cleaner to make code compatible with jdk11

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

xiangfu pushed a commit to branch remove_misc_cleaner_ref
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git

commit e1d9cfb67bfe85eae74f687240fccda1fe807b85
Author: Xiang Fu <fx...@gmail.com>
AuthorDate: Thu Sep 19 17:15:22 2019 +0800

    Remove import of sun.misc.Cleaner to make code compatible with jdk11
---
 .../java/org/apache/pinot/core/segment/memory/PinotByteBuffer.java  | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/pinot-core/src/main/java/org/apache/pinot/core/segment/memory/PinotByteBuffer.java b/pinot-core/src/main/java/org/apache/pinot/core/segment/memory/PinotByteBuffer.java
index 956dbc2..21a51ad 100644
--- a/pinot-core/src/main/java/org/apache/pinot/core/segment/memory/PinotByteBuffer.java
+++ b/pinot-core/src/main/java/org/apache/pinot/core/segment/memory/PinotByteBuffer.java
@@ -26,7 +26,6 @@ import java.nio.ByteOrder;
 import java.nio.MappedByteBuffer;
 import java.nio.channels.FileChannel;
 import javax.annotation.concurrent.ThreadSafe;
-import sun.misc.Cleaner;
 import sun.nio.ch.DirectBuffer;
 
 
@@ -328,9 +327,8 @@ public class PinotByteBuffer extends PinotDataBuffer {
 
   @Override
   protected void release() {
-    Cleaner cleaner = ((DirectBuffer) _buffer).cleaner();
-    if (cleaner != null) {
-      cleaner.clean();
+    if (((DirectBuffer) _buffer).cleaner() != null) {
+      ((DirectBuffer) _buffer).cleaner().clean();
     }
   }
 }


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