Remaining two default overloads of execute/executeAsync, and remove redundant Impl implementations.

Author: GregoryTravis

temporal-sdk/src/main/java/io/temporal/client/ActivityClient.java

@@ -1453,7 +1453,10 @@ <R> ActivityHandle<R> start(
    * @param args activity arguments
    * @throws ActivityFailedException if the activity failed, timed out, or was cancelled
    */
-  void execute(String activityType, StartActivityOptions options, @Nullable Object... args);
+  default void execute(
+      String activityType, StartActivityOptions options, @Nullable Object... args) {
+    start(activityType, options, args).getResult(Void.class);
+  }
 
   /**
    * Starts a standalone activity and blocks until it completes, returning the typed result.
@@ -1504,8 +1507,10 @@ default <R> R execute(
    * @param args activity arguments
    * @return a future that completes when the activity completes
    */
-  CompletableFuture<Void> executeAsync(
-      String activityType, StartActivityOptions options, @Nullable Object... args);
+  default CompletableFuture<Void> executeAsync(
+      String activityType, StartActivityOptions options, @Nullable Object... args) {
+    return start(activityType, options, args).getResultAsync(Void.class);
+  }
 
   /**
    * Starts a standalone activity and returns a future that completes with the typed result.

temporal-sdk/src/main/java/io/temporal/client/ActivityClientImpl.java

@@ -20,7 +20,6 @@
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
-import java.util.concurrent.CompletableFuture;
 import java.util.stream.Stream;
 import javax.annotation.Nullable;
 
@@ -318,60 +317,6 @@ public <R> ActivityHandle<R> start(
     return ActivityHandle.fromUntyped(untyped, resultClass, resultType);
   }
 
-  // ---- String-based execute ----
-
-  @Override
-  @SuppressWarnings("unchecked")
-  public void execute(String activityType, StartActivityOptions options, @Nullable Object... args) {
-    start(activityType, options, args).getResult(Void.class);
-  }
-
-  @Override
-  public <R> R execute(
-      String activityType,
-      Class<R> resultClass,
-      StartActivityOptions options,
-      @Nullable Object... args) {
-    return start(activityType, resultClass, options, args).getResult();
-  }
-
-  @Override
-  public <R> R execute(
-      String activityType,
-      Class<R> resultClass,
-      Type resultType,
-      StartActivityOptions options,
-      @Nullable Object... args) {
-    return start(activityType, resultClass, resultType, options, args).getResult();
-  }
-
-  // ---- String-based executeAsync ----
-
-  @Override
-  public CompletableFuture<Void> executeAsync(
-      String activityType, StartActivityOptions options, @Nullable Object... args) {
-    return start(activityType, options, args).getResultAsync(Void.class);
-  }
-
-  @Override
-  public <R> CompletableFuture<R> executeAsync(
-      String activityType,
-      Class<R> resultClass,
-      StartActivityOptions options,
-      @Nullable Object... args) {
-    return start(activityType, resultClass, options, args).getResultAsync();
-  }
-
-  @Override
-  public <R> CompletableFuture<R> executeAsync(
-      String activityType,
-      Class<R> resultClass,
-      Type resultType,
-      StartActivityOptions options,
-      @Nullable Object... args) {
-    return start(activityType, resultClass, resultType, options, args).getResultAsync();
-  }
-
   // ---- Handle lookup ----
 
   @Override