Throw exception on unsuccessfull NewGlobalRef call.

facebook · Jul 9, 2024 · 5887ad0 · 5887ad0
1 parent fe5549c
commit 5887ad0
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/java/rocksjni/jni_merge_operator_v2.cc b/java/rocksjni/jni_merge_operator_v2.cc
@@ -57,7 +57,11 @@ JniMergeOperatorV2::JniMergeOperatorV2(JNIEnv* env, jobject java_merge_operator,
   }
   j_merge_class = static_cast<jclass>(env->NewGlobalRef(j_merge_class));
   if (j_merge_class == nullptr) {
-    return;  // Exception
+    if(env->ExceptionCheck() == JNI_FALSE) {
+      RocksDBExceptionJni::ThrowNew(
+          env, "Unable to obtain GlobalRef for merge operator");
+    }
+    return;
   }
 
   j_merge_internal =