From e0731a3726a25ad384c7cbbfa37208b975c6b568 Mon Sep 17 00:00:00 2001
From: Hansong Zhang <hsz@meta.com>
Date: Wed, 17 Apr 2024 16:54:20 -0700
Subject: [PATCH] throw Java exception when execution fails

Summary: Instead of logging, we throw a java exception and let user catch it.

Reviewed By: dbcakadbc

Differential Revision: D56270287
---
 extension/android/jni/jni_layer.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/extension/android/jni/jni_layer.cpp b/extension/android/jni/jni_layer.cpp
index 65590e7d43e..ac93f5a7137 100644
--- a/extension/android/jni/jni_layer.cpp
+++ b/extension/android/jni/jni_layer.cpp
@@ -325,8 +325,8 @@ class ExecuTorchJni : public facebook::jni::HybridClass<ExecuTorchJni> {
 #endif
 
     if (!result.ok()) {
-      ET_LOG(
-          Error,
+      facebook::jni::throwNewJavaException(
+          "java/lang/Exception",
           "Execution of method %s failed with status 0x%" PRIx32,
           method.c_str(),
           static_cast<error_code_t>(result.error()));