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

Skip to content

[bug] fix the pagination issue in checkpointBatcher#54

Merged
zhongkechen merged 1 commit intomainfrom
fix/checkpoint
Feb 5, 2026
Merged

[bug] fix the pagination issue in checkpointBatcher#54
zhongkechen merged 1 commit intomainfrom
fix/checkpoint

Conversation

@zhongkechen
Copy link
Contributor

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

#53

Description

  • bug: Checkpoint call wasn't followed by a GetExecutionState API call to fetch the following pages if nextMarker isn't empty
  • refactor: move checkpointToken into CheckpointBatcher

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? Wonder how this could be tested

Integration Tests

Have integration tests been written for these changes? Wonder how this could be tested

Examples

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

@zhongkechen zhongkechen marked this pull request as ready for review February 5, 2026 01:33
@phipag phipag linked an issue Feb 5, 2026 that may be closed by this pull request
@zhongkechen zhongkechen merged commit 8110ddc into main Feb 5, 2026
6 of 7 checks passed
@zhongkechen zhongkechen deleted the fix/checkpoint branch February 5, 2026 17:29
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.

[Bug]: checkpoint call doesn't fetch paginated operations

2 participants