Skip to content

Conversation

@mjameswh
Copy link
Contributor

Description

Remove the following:

  • IHandler interface:

    • FetchOperationResultAsync(OperationFetchResultContext context)
    • FetchOperationInfoAsync(OperationFetchInfoContext context)
  • IOperationHandler<TInput, TResult> interface:

    • FetchResultAsync(OperationFetchResultContext context)
    • FetchInfoAsync(OperationFetchInfoContext context)
  • OperationFetchResultContext.cs - Context class for fetch result operations

  • OperationFetchInfoContext.cs - Context class for fetch info operations

  • OperationInfo.cs - Information about an operation

  • OperationStillRunningException.cs - Exception for when operation is still running

Fixes #10

@mjameswh mjameswh requested a review from a team as a code owner October 17, 2025 20:58
Copy link
Contributor

@cretz cretz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved, but I think we don't want to merge until we have a release plan for the Temporal side

@mjameswh mjameswh changed the title Remove FetchInfo and FetchResult requests 💥 Remove FetchInfo and FetchResult requests Oct 17, 2025
@mjameswh mjameswh merged commit 98c5ba8 into main Oct 21, 2025
3 checks passed
@mjameswh mjameswh deleted the remove-handler-getinfo branch October 21, 2025 22:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove FetchInfo and FetchResult requests

4 participants