Draft
Conversation
This was referenced Feb 27, 2026
feiyang3cat
reviewed
Mar 20, 2026
| google.protobuf.Timestamp auto_skip_deadline = 115; | ||
|
|
||
| // Number of auto-skip firings consumed in the current window. | ||
| int32 auto_skip_firings_used = 116; |
Contributor
There was a problem hiding this comment.
choose not to include firing count related control at initial version of the feature as we may choose to optimize the time-firing logic which makes customers confused when they try to count how many timepoints to skip
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed?
Added implementation for controlling workflow time skipping.
DO NOT MERGE - this is more for review/visibility only, and much of it was built without deep understanding of server details, even if it does technically work well. Server team is better equipped to take this PR as a PoC and do what they see as best.
API PR
temporalio/api#725
SDK PR
temporalio/sdk-python#1345