Skip to content

add time skipping to (both chasm-based and wf-based) schedules - #11756

Closed
feiyang3cat wants to merge 3 commits into
temporalio:fx/vts-chasm-frameworkfrom
feiyang3cat:fx/vts-schedule
Closed

add time skipping to (both chasm-based and wf-based) schedules#11756
feiyang3cat wants to merge 3 commits into
temporalio:fx/vts-chasm-frameworkfrom
feiyang3cat:fx/vts-schedule

Conversation

@feiyang3cat

@feiyang3cat feiyang3cat commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

What changed?

add time skipping to schedules (CHASM-based and workflow-based)

Stack

How did you test it?

  • built
  • run locally and tested manually
  • covered by existing tests
  • added new unit test(s)
  • added new functional test(s)

@feiyang3cat feiyang3cat changed the title add time skipping to schedules (chasm based and wf based) add time skipping to schedules (both chasm-based and wf-based) Aug 24, 2026
@feiyang3cat feiyang3cat changed the title add time skipping to schedules (both chasm-based and wf-based) add time skipping to (both chasm-based and wf-based) schedules Aug 24, 2026
@@ -351,6 +354,31 @@ func (s *Scheduler) LifecycleState(ctx chasm.Context) chasm.LifecycleState {
return chasm.LifecycleStateRunning
}

@feiyang3cat feiyang3cat Aug 24, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

by default:

time-skipping provided by the chasm framework scans the whole component tree for valid timer tasks to skip

  1. IsExecutionSkippable can be a harness of this behavior
  2. a stronger harness is to override the default timer-picking behavior and add specific logic for schedules

@feiyang3cat

Copy link
Copy Markdown
Contributor Author

Superseded by #11761, which uses the upstream fx/vts-schedule branch as the PR head.

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