Deprecation of compose.GetState, Use compose.ProcessState instead #323
Closed
hi-pender
announced in
Announcements
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
To enhance execution efficiency and concurrency safety in Eino Graph's runner engine, we're removing
compose.GetState
effective immediately. This change is necessary due to fundamental differences in our execution models:Background
Eino's runner supports two distinct execution modes:
Requires explicit
SuperStep
synchronization for vertex computation schedulingDesigned for true parallel execution without step synchronization
The Problem
During initial implementation, DAG mode reused parts of the Pregel runner, inadvertently inheriting its
SuperStep
model. This creates artificial synchronization barriers that:So we'll remove DAG's
SuperStep
, and it'll cause concurrency safety issues withcompose.GetState
Solution
We're removing the non-concurrency-safe
compose.GetState
and eliminatingSuperStep
artifacts from DAG execution. All implementations must migrate to the replacement API:Key Benefits
Migration Timeline
compose.GetState
in new codeActions Required
Replace all instances of:
with:
Beta Was this translation helpful? Give feedback.
All reactions