Skip to content

Conversation

@mjameswh
Copy link
Contributor

Description

Remove the following:

  • FetchInfo and FetchResult requests from AbstractHandler and Handler classes
  • fetch_info and fetch_result methods from OperationHandler abstract class
  • fetch_info and fetch_result methods from both SyncOperationHandler classes
  • FetchOperationInfoContext and FetchOperationResultContext classes
  • OperationInfo class and its associated OperationState enum

Fix #23


__all__ = [
"CancelOperationContext",
"FetchOperationInfoContext",
Copy link

Choose a reason for hiding this comment

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

Mentioned in .NET but will also mention here. This is technically an incompatible change to remove a type that SDKs are using. Can you put 💥 on the issue title? Also, we need to make sure not to merge this until we have a plan to release SDK and Nexus at the same time, otherwise latest SDK won't compile with latest Nexus if this is released first.

@mjameswh mjameswh changed the title Remove FetchInfo and FetchResult handler requests 💥 Remove FetchInfo and FetchResult handler requests Oct 17, 2025
@mjameswh mjameswh merged commit 5c0435e into main Oct 27, 2025
10 checks passed
@mjameswh mjameswh deleted the remove-fetch-info branch October 27, 2025 14:46
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