Implementation of Fault tolerance 4.0

fault-tolerance/src/main/java/io/helidon/faulttolerance/RetryImpl.java

@@ -73,9 +73,10 @@ private <T> Single<T> retrySingle(RetryContext<? extends CompletionStage<T>> con
 
         long nanos = System.nanoTime() - context.startedNanos;
         if (nanos > maxTimeNanos) {
-            TimeoutException te = new TimeoutException("Execution took too long. Already executing: "
-                    + TimeUnit.NANOSECONDS.toMillis(nanos) + " ms, must timeout after: "
-                    + TimeUnit.NANOSECONDS.toMillis(maxTimeNanos) + " ms.");
+            TimeoutException te = new RetryTimeoutException(context.throwable(),
+                    "Execution took too long. Already executing: "
+                            + TimeUnit.NANOSECONDS.toMillis(nanos) + " ms, must timeout after: "
+                            + TimeUnit.NANOSECONDS.toMillis(maxTimeNanos) + " ms.");
             if (context.hasThrowable()) {
                 te.initCause(context.throwable());
             }
@@ -115,9 +116,10 @@ private <T> Multi<T> retryMulti(RetryContext<? extends Flow.Publisher<T>> contex
 
         long nanos = System.nanoTime() - context.startedNanos;
         if (nanos > maxTimeNanos) {
-            return Multi.error(new TimeoutException("Execution took too long. Already executing: "
-                                                             + TimeUnit.NANOSECONDS.toMillis(nanos) + " ms, must timeout after: "
-                                                             + TimeUnit.NANOSECONDS.toMillis(maxTimeNanos) + " ms."));
+            return Multi.error(new RetryTimeoutException(context.throwable(),
+                    "Execution took too long. Already executing: "
+                            + TimeUnit.NANOSECONDS.toMillis(nanos) + " ms, must timeout after: "
+                            + TimeUnit.NANOSECONDS.toMillis(maxTimeNanos) + " ms."));
         }
 
         if (currentCallIndex > 0) {
@@ -190,3 +192,4 @@ Throwable throwable() {
         }
     }
 }
+

fault-tolerance/src/main/java/io/helidon/faulttolerance/RetryTimeoutException.java

@@ -0,0 +1,51 @@
+/*
+ * Copyright (c) 2021 Oracle and/or its affiliates.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package io.helidon.faulttolerance;
+
+import java.util.concurrent.TimeoutException;
+
+/**
+ * Subclass of {@link TimeoutException} to discern exceptions thrown by a {@link Retry}
+ * when its overall timeout is reached versus those thrown by a {@link Timeout}.
+ */
+public class RetryTimeoutException extends TimeoutException {
+    private static final long serialVersionUID = 1900926677490550714L;
+
+    private final Throwable lastRetryException;
+
+    /**
+     * Constructs a {@code RetryTimeoutException} with the specified detail
+     * message.
+     *
+     * @param throwable last retry exception
+     * @param message the detail message
+     */
+    public RetryTimeoutException(Throwable throwable, String message) {
+        super(message);
+        lastRetryException = throwable;
+    }
+
+    /**
+     * Last exception thrown in {@code Retry} before the overall timeout reached.
+     *
+     * @return last exception thrown
+     */
+    public Throwable lastRetryException() {
+        return lastRetryException;
+    }
+}
+

jersey/client/src/main/resources/META-INF/native-image/io.helidon.jersey/helidon-jersey-client/native-image.properties

@@ -0,0 +1,17 @@
+#
+# Copyright (c) 2021 Oracle and/or its affiliates.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+Args=--initialize-at-run-time=org.glassfish.jersey.client.internal.HttpUrlConnector

microprofile/fault-tolerance/etc/spotbugs/exclude.xml

@@ -0,0 +1,34 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!--
+
+    Copyright (c) 2021 Oracle and/or its affiliates.
+
+    Licensed under the Apache License, Version 2.0 (the "License");
+    you may not use this file except in compliance with the License.
+    You may obtain a copy of the License at
+
+        http://www.apache.org/licenses/LICENSE-2.0
+
+    Unless required by applicable law or agreed to in writing, software
+    distributed under the License is distributed on an "AS IS" BASIS,
+    WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+    See the License for the specific language governing permissions and
+    limitations under the License.
+
+-->
+
+<FindBugsFilter
+        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+        xmlns="https://github.com/spotbugs/filter/3.0.0"
+        xsi:schemaLocation="https://github.com/spotbugs/filter/3.0.0 https://raw.githubusercontent.com/spotbugs/spotbugs/3.1.0/spotbugs/etc/findbugsfilter.xsd">
+
+    <Match>
+        <!--
+          - Incorrectly reported on just one enum in FaultToleranceMetrics.java
+          - https://github.com/spotbugs/spotbugs/issues/740
+          -->
+        <Field type="org.eclipse.microprofile.metrics.Tag" />
+        <Bug pattern="SE_BAD_FIELD" />
+    </Match>
+
+</FindBugsFilter>

microprofile/fault-tolerance/pom.xml

@@ -33,6 +33,10 @@
         Microprofile fault tolerance implementation
     </description>
 
+    <properties>
+        <spotbugs.exclude>etc/spotbugs/exclude.xml</spotbugs.exclude>
+    </properties>
+
     <build>
         <plugins>
             <plugin>

microprofile/fault-tolerance/src/main/java/io/helidon/microprofile/faulttolerance/CommandFallback.java → microprofile/fault-tolerance/src/main/java/io/helidon/microprofile/faulttolerance/FallbackHelper.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2018, 2020 Oracle and/or its affiliates.
+ * Copyright (c) 2018, 2021 Oracle and/or its affiliates.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -26,9 +26,9 @@
 import org.eclipse.microprofile.faulttolerance.FallbackHandler;
 
 /**
- * Class CommandFallback.
+ * Implements invocation callback logic.
  */
-class CommandFallback {
+class FallbackHelper {
 
     private final InvocationContext context;
 
@@ -45,7 +45,7 @@ class CommandFallback {
      * @param introspector Method introspector.
      * @param throwable Throwable that caused execution of fallback
      */
-    CommandFallback(InvocationContext context, MethodIntrospector introspector, Throwable throwable) {
+    FallbackHelper(InvocationContext context, MethodIntrospector introspector, Throwable throwable) {
         this.context = context;
         this.throwable = throwable;
 
@@ -103,24 +103,13 @@ public Throwable getFailure() {
                 result = fallbackMethod.invoke(context.getTarget(), context.getParameters());
             }
         } catch (Throwable t) {
-            updateMetrics();
-
             // If InvocationTargetException, then unwrap underlying cause
             if (t instanceof InvocationTargetException) {
                 t = t.getCause();
             }
             throw t instanceof Exception ? (Exception) t : new RuntimeException(t);
         }
 
-        updateMetrics();
         return result;
     }
-
-    /**
-     * Updates fallback metrics.
-     */
-    private void updateMetrics() {
-        Method method = context.getMethod();
-        FaultToleranceMetrics.getCounter(method, FaultToleranceMetrics.FALLBACK_CALLS_TOTAL).inc();
-    }
 }

microprofile/fault-tolerance/src/main/java/io/helidon/microprofile/faulttolerance/FaultToleranceExtension.java

@@ -229,38 +229,29 @@ private void registerFaultToleranceMethods(AnnotatedType<?> type) {
     }
 
     /**
-     * Registers metrics for all FT methods and init executors.
+     * Validates annotations.
      *
      * @param validation Event information.
      */
-    void registerMetricsAndInitExecutors(@Observes AfterDeploymentValidation validation) {
+    void validateAnnotations(@Observes AfterDeploymentValidation validation) {
         if (FaultToleranceMetrics.enabled()) {
             getRegisteredMethods().stream().forEach(beanMethod -> {
                 final Method method = beanMethod.method();
                 final Class<?> beanClass = beanMethod.beanClass();
 
-                // Counters for all methods
-                FaultToleranceMetrics.registerMetrics(method);
-
-                // Metrics depending on the annotationSet present
                 if (MethodAntn.isAnnotationPresent(beanClass, method, Retry.class)) {
-                    FaultToleranceMetrics.registerRetryMetrics(method);
                     new RetryAntn(beanClass, method).validate();
                 }
                 if (MethodAntn.isAnnotationPresent(beanClass, method, CircuitBreaker.class)) {
-                    FaultToleranceMetrics.registerCircuitBreakerMetrics(method);
                     new CircuitBreakerAntn(beanClass, method).validate();
                 }
                 if (MethodAntn.isAnnotationPresent(beanClass, method, Timeout.class)) {
-                    FaultToleranceMetrics.registerTimeoutMetrics(method);
                     new TimeoutAntn(beanClass, method).validate();
                 }
                 if (MethodAntn.isAnnotationPresent(beanClass, method, Bulkhead.class)) {
-                    FaultToleranceMetrics.registerBulkheadMetrics(method);
                     new BulkheadAntn(beanClass, method).validate();
                 }
                 if (MethodAntn.isAnnotationPresent(beanClass, method, Fallback.class)) {
-                    FaultToleranceMetrics.registerFallbackMetrics(method);
                     new FallbackAntn(beanClass, method).validate();
                 }
                 if (MethodAntn.isAnnotationPresent(beanClass, method, Asynchronous.class)) {
@@ -269,17 +260,17 @@ void registerMetricsAndInitExecutors(@Observes AfterDeploymentValidation validat
             });
         }
 
-        // Initialize executors for MP FT - default size of 16
+        // Initialize executors for MP FT - default size of 20
         io.helidon.config.Config config = MpConfig.toHelidonConfig(ConfigProvider.getConfig());
         scheduledThreadPoolSupplier = ScheduledThreadPoolSupplier.builder()
                 .threadNamePrefix("ft-mp-schedule-")
-                .corePoolSize(16)
+                .corePoolSize(20)
                 .config(config.get("scheduled-executor"))
                 .build();
         FaultTolerance.scheduledExecutor(scheduledThreadPoolSupplier);
         threadPoolSupplier = ThreadPoolSupplier.builder()
                 .threadNamePrefix("ft-mp-")
-                .corePoolSize(16)
+                .corePoolSize(20)
                 .config(config.get("executor"))
                 .build();
         FaultTolerance.executor(threadPoolSupplier);