Removes recordFailure and getFailure methods which were previously de…

…precated.

CHANGELOG.md

@@ -3,6 +3,7 @@
 ### API Changes
 
 - `ExecutionContext.getStartTime` now returns an `Instant` rather than a `Duration` object.
+- `ExecutionContext.getLastFailure`, `Execution.recordFailure` and similar methods for recording Exceptions, which were previously deprecated, were removed. Use `getLastException`, `recordException`, etc. instead.
 
 # 3.2.4
 

core/src/main/java/dev/failsafe/AsyncExecution.java

@@ -61,10 +61,4 @@ public interface AsyncExecution<R> extends ExecutionContext<R> {
    * @throws IllegalStateException if the most recent execution was already recorded or the execution is complete
    */
   void recordException(Throwable exception);
-
-  /**
-   * @deprecated Use {@link #recordException(Throwable)} instead
-   */
-  @Deprecated
-  void recordFailure(Throwable failure);
 }

core/src/main/java/dev/failsafe/AsyncExecutionImpl.java

@@ -111,12 +111,6 @@ public void recordException(Throwable exception) {
     record(null, exception);
   }
 
-  @Override
-  @Deprecated
-  public void recordFailure(Throwable failure) {
-    recordException(failure);
-  }
-
   @Override
   public boolean isAsyncExecution() {
     return asyncExecution;

core/src/main/java/dev/failsafe/CircuitBreaker.java

@@ -233,12 +233,6 @@ default void acquirePermit() {
    */
   void recordException(Throwable exception);
 
-  /**
-   * @deprecated Use {@link #recordException(Throwable)} instead.
-   */
-  @Deprecated
-  void recordFailure(Throwable failure);
-
   /**
    * Records an execution {@code result} as a success or failure based on the failure configuration.
    */

core/src/main/java/dev/failsafe/Execution.java

@@ -81,10 +81,4 @@ static <R> Execution<R> of(Policy<R> outerPolicy, Policy<R>... policies) {
    * @throws IllegalStateException if the execution is already complete
    */
   void recordException(Throwable exception);
-
-  /**
-   * @deprecated Use {@link #recordException(Throwable)} instead
-   */
-  @Deprecated
-  void recordFailure(Throwable failure);
 }

core/src/main/java/dev/failsafe/ExecutionContext.java

@@ -62,12 +62,6 @@ public interface ExecutionContext<R> {
    */
   <T extends Throwable> T getLastException();
 
-  /**
-   * @deprecated Use {@link #getLastException()} instead
-   */
-  @Deprecated
-  <T extends Throwable> T getLastFailure();
-
   /**
    * Returns the last result that was recorded else {@code null}.
    */

core/src/main/java/dev/failsafe/ExecutionImpl.java

@@ -244,12 +244,6 @@ public <T extends Throwable> T getLastException() {
     return r == null ? null : (T) r.getException();
   }
 
-  @Override
-  @Deprecated
-  public <T extends Throwable> T getLastFailure() {
-    return getLastException();
-  }
-
   @Override
   public R getLastResult() {
     ExecutionResult<R> r = result != null ? result : previousResult;

core/src/main/java/dev/failsafe/SyncExecutionImpl.java

@@ -132,12 +132,6 @@ public void recordException(Throwable exception) {
     postExecute(new ExecutionResult<>(null, exception));
   }
 
-  @Override
-  @Deprecated
-  public void recordFailure(Throwable failure) {
-    recordException(failure);
-  }
-
   @Override
   public synchronized void preExecute() {
     if (isStandalone()) {

core/src/main/java/dev/failsafe/event/ExecutionAttemptedEvent.java

@@ -40,14 +40,6 @@ public Throwable getLastException() {
     return exception;
   }
 
-  /**
-   * @deprecated Use {@link #getLastException()} instead
-   */
-  @Deprecated
-  public Throwable getLastFailure() {
-    return exception;
-  }
-
   /**
    * Returns the result that preceded the event, else {@code null} if there was none.
    */

core/src/main/java/dev/failsafe/event/ExecutionCompletedEvent.java

@@ -40,14 +40,6 @@ public Throwable getException() {
     return exception;
   }
 
-  /**
-   * @deprecated Use {@link #getException()} instead
-   */
-  @Deprecated
-  public Throwable getFailure() {
-    return exception;
-  }
-
   /**
    * Returns the result that preceded the event, else {@code null} if there was none.
    */

core/src/main/java/dev/failsafe/event/ExecutionScheduledEvent.java

@@ -49,14 +49,6 @@ public Throwable getLastException() {
     return exception;
   }
 
-  /**
-   * @deprecated Use {@link #getLastException()} instead
-   */
-  @Deprecated
-  public Throwable getLastFailure() {
-    return exception;
-  }
-
   /**
    * Returns the result that preceded the event, else {@code null} if there was none.
    */

core/src/main/java/dev/failsafe/internal/CircuitBreakerImpl.java

@@ -129,12 +129,6 @@ public void recordException(Throwable exception) {
     recordResult(null, exception);
   }
 
-  @Override
-  @Deprecated
-  public void recordFailure(Throwable failure) {
-    recordException(failure);
-  }
-
   @Override
   public void recordResult(R result) {
     recordResult(result, null);