Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[refactor] remove redundant InitialExecutionState#75

Merged
zhongkechen merged 1 commit intomainfrom
InitialExecutionState
Feb 12, 2026
Merged

[refactor] remove redundant InitialExecutionState#75
zhongkechen merged 1 commit intomainfrom
InitialExecutionState

Conversation

@zhongkechen
Copy link
Contributor

@zhongkechen zhongkechen commented Feb 12, 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

  • removed the redundant InitialExecutionState class. Lambda API already provided a class CheckpointUpdatedExecutionState for the same purpose.
  • updated fetchAllPages to work with CheckpointUpdatedExecutionState from the invoke payload as well as the getExecutionState response

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? updated

Examples

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

@zhongkechen zhongkechen marked this pull request as ready for review February 12, 2026 05:59
@zhongkechen zhongkechen merged commit c45eac8 into main Feb 12, 2026
9 of 10 checks passed
@zhongkechen zhongkechen deleted the InitialExecutionState branch February 25, 2026 06:51
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