Remove FetchInfo and FetchResult handler requests (#40)

Author: mjameswh

nexus-sdk/src/main/java/io/nexusrpc/OperationInfo.java

@@ -1,9 +1,6 @@
 package io.nexusrpc.handler;
 
-import io.nexusrpc.Experimental;
 import io.nexusrpc.OperationException;
-import io.nexusrpc.OperationInfo;
-import io.nexusrpc.OperationStillRunningException;
 
 /** Top-level handler for service calls. */
 public interface Handler {
@@ -18,25 +15,6 @@ OperationStartResult<HandlerResultContent> startOperation(
       OperationContext context, OperationStartDetails details, HandlerInputContent input)
       throws OperationException, HandlerException;
 
-  /**
-   * Fetch the result for an asynchronously started operation. See {@link
-   * OperationHandler#fetchResult} for operation details.
-   *
-   * <p>If the result is an output stream, it will be closed later by the caller.
-   */
-  @Experimental
-  HandlerResultContent fetchOperationResult(
-      OperationContext context, OperationFetchResultDetails details)
-      throws OperationStillRunningException, OperationException, HandlerException;
-
-  /**
-   * Fetch information about the asynchronously started operation. See {@link
-   * OperationHandler#fetchInfo} for details.
-   */
-  @Experimental
-  OperationInfo fetchOperationInfo(OperationContext context, OperationFetchInfoDetails details)
-      throws HandlerException;
-
   /**
    * Cancel the asynchronously started operation. See {@link OperationHandler#cancel} for details.
    */

nexus-sdk/src/main/java/io/nexusrpc/OperationStillRunningException.java

@@ -1,8 +1,6 @@
 package io.nexusrpc.handler;
 
 import io.nexusrpc.OperationException;
-import io.nexusrpc.OperationInfo;
-import io.nexusrpc.OperationStillRunningException;
 import org.jspecify.annotations.Nullable;
 
 /**
@@ -47,41 +45,6 @@ OperationStartResult<R> start(
       OperationContext context, OperationStartDetails details, @Nullable T param)
       throws OperationException, HandlerException;
 
-  /**
-   * Fetch the result for an asynchronously started operation.
-   *
-   * @param context Context for the call.
-   * @param details Details for the call including the operation token. The details also contain a
-   *     timeout which affects how implementers should implement this function. See {@link
-   *     OperationFetchResultDetails#getTimeout()} to see how to react to this value.
-   * @return The resulting value upon success.
-   * @throws OperationStillRunningException Operation is still running beyond the given timeout.
-   * @throws OperationException Operation failed. If thrown, can have failure details and state such
-   *     as saying the operation was cancelled.
-   * @throws HandlerException Unexpected failures while running the handler. This should be thrown
-   *     with a type of {@link HandlerException.ErrorType#NOT_FOUND} if the operation token is not
-   *     found.
-   * @throws RuntimeException Any other exception, will be converted to an {@link HandlerException}
-   *     of type {@link HandlerException.ErrorType#INTERNAL}.
-   */
-  @Nullable R fetchResult(OperationContext context, OperationFetchResultDetails details)
-      throws OperationStillRunningException, OperationException, HandlerException;
-
-  /**
-   * Fetch information about the asynchronously started operation.
-   *
-   * @param context Context for the call.
-   * @param details Details for the call including the operation token.
-   * @return Information about the operation.
-   * @throws HandlerException Unexpected failures while running the handler. This should be thrown
-   *     with a type of {@link HandlerException.ErrorType#NOT_FOUND} if the operation token is not
-   *     found.
-   * @throws RuntimeException Any other exception, will be converted to an {@link HandlerException}
-   *     of type {@link HandlerException.ErrorType#INTERNAL}.
-   */
-  OperationInfo fetchInfo(OperationContext context, OperationFetchInfoDetails details)
-      throws HandlerException;
-
   /**
    * Cancel the asynchronously started operation.
    *