Skip to content

[refactor]: move serdes out of DurableHandler#225

Merged
zhongkechen merged 1 commit intomainfrom
input_output_serdes
Mar 18, 2026
Merged

[refactor]: move serdes out of DurableHandler#225
zhongkechen merged 1 commit intomainfrom
input_output_serdes

Conversation

@zhongkechen
Copy link
Contributor

@zhongkechen zhongkechen commented Mar 18, 2026

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Issue Link, if available

Description

clean up implementation details in DurableHandler to prevent misuse

  • moved objectMapper in DurableHandler to a new dedicated class for ser/des
  • moved validateConfiguration to DurableConfig

Demo/Screenshots

Checklist

  • I have filled out every section of the PR template
  • I have thoroughly tested this change

Testing

Unit Tests

Have unit tests been written for these changes? Updated

Integration Tests

Have integration tests been written for these changes?

Examples

Has a new example been added for the change? (if applicable)

@zhongkechen zhongkechen marked this pull request as ready for review March 18, 2026 00:29
@zhongkechen zhongkechen requested a review from a team March 18, 2026 00:29
@zhongkechen zhongkechen force-pushed the input_output_serdes branch from 6d5f351 to 27e173c Compare March 18, 2026 00:34
@zhongkechen zhongkechen merged commit 5776bd5 into main Mar 18, 2026
11 checks passed
@zhongkechen zhongkechen deleted the input_output_serdes branch March 18, 2026 02:12
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.

2 participants