Skip to content

Add support for serialization type hints - #3081

Draft
Quinn-With-Two-Ns wants to merge 2 commits into
temporalio:mainfrom
Quinn-With-Two-Ns:support-type-hints
Draft

Quinn-With-Two-Ns wants to merge 2 commits into
temporalio:mainfrom
Quinn-With-Two-Ns:support-type-hints

Conversation

@Quinn-With-Two-Ns

Copy link
Copy Markdown
Contributor

What was changed

Why?

Checklist

  1. Closes

  2. How was this tested:

  1. Any docs updates needed?

Named type-hint entry points avoid Java overload resolution treating a null first workflow argument as the Type[] parameter while retaining default implementations for existing WorkflowStub decorators.

Constraint: Public interface additions require default implementations for compatibility
Rejected: Keep Type[] overloads | null workflow arguments bind to the more-specific overload and shift serialized arguments
Confidence: high
Scope-risk: narrow
Tested: DescribeWorkflowExecutionTest.testChildWorkflow; SerializationTypeHintTest; StartTest; CustomDataConverterTest; spotlessCheck; assemble
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.

1 participant