Skip to content

perf(scheduler): stop re-encoding the session config for every task - #2358

Open
NoahKusaba wants to merge 1 commit into
apache:mainfrom
NoahKusaba:perf/hoist-task-session-props
Open

perf(scheduler): stop re-encoding the session config for every task#2358
NoahKusaba wants to merge 1 commit into
apache:mainfrom
NoahKusaba:perf/hoist-task-session-props

Conversation

@NoahKusaba

@NoahKusaba NoahKusaba commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Closes #.

Rationale for this change

Release build, 32-task batch, 200 iterations, measured on
prepare_multi_task_definition end to end:

per batch per task
before 3.53 ms 110.5 µs
after 1.77 ms 55.5 µs

What changes are included in this PR?

Encode the session config once per task batch instead of once per task, cloning it into each task's definition.

@NoahKusaba
NoahKusaba force-pushed the perf/hoist-task-session-props branch from b4f787a to 8025374 Compare August 23, 2026 16:59
@NoahKusaba
NoahKusaba force-pushed the perf/hoist-task-session-props branch from 8025374 to 63421e8 Compare August 23, 2026 17:05
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