Remove workaround for JDK-8059299

trinodb · Oct 24, 2024 · f9f6943 · f9f6943
1 parent af625c5
commit f9f6943
Showing 1 changed file with 1 addition and 10 deletions.
diff --git a/...ino-hive/src/main/java/io/trino/plugin/hive/metastore/thrift/ThriftMetastoreApiStats.java b/...ino-hive/src/main/java/io/trino/plugin/hive/metastore/thrift/ThriftMetastoreApiStats.java
@@ -43,23 +43,14 @@ public <V> Callable<V> wrap(Callable<V> callable)
             catch (Exception e) {
                 if (e instanceof MetaException) {
                     metastoreExceptions.update(1);
-                    // Need to throw here instead of falling through due to JDK-8059299
-                    totalFailures.update(1);
-                    throw e;
                 }
-
-                if (e instanceof TException) {
+                else if (e instanceof TException) {
                     if (e instanceof TBase) {
                         // This exception is an API response and not a server error
                         throw e;
                     }
-
                     thriftExceptions.update(1);
-                    // Need to throw here instead of falling through due to JDK-8059299
-                    totalFailures.update(1);
-                    throw e;
                 }
-
                 totalFailures.update(1);
                 throw e;
             }