-
Notifications
You must be signed in to change notification settings - Fork 1.8k
[sbc] StateBackedComponent class #31424
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
OwenKephart
wants to merge
1
commit into
benpankow/persist-load-context
Choose a base branch
from
07-29-_sbc_statebackedcomponent_class
base: benpankow/persist-load-context
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
[sbc] StateBackedComponent class #31424
OwenKephart
wants to merge
1
commit into
benpankow/persist-load-context
from
07-29-_sbc_statebackedcomponent_class
Conversation
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
This was referenced Jul 30, 2025
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
9cd14f2
to
b54a941
Compare
d3d155a
to
4cc1db0
Compare
python_modules/dagster/dagster/components/component/state_backed_component.py
Outdated
Show resolved
Hide resolved
b54a941
to
af35cdb
Compare
4cc1db0
to
3391773
Compare
python_modules/dagster/dagster/components/component/state_backed_component.py
Outdated
Show resolved
Hide resolved
3391773
to
928ca52
Compare
af35cdb
to
3352ae0
Compare
2d9f5d3
to
daa5054
Compare
3352ae0
to
ac8f7bc
Compare
ac8f7bc
to
e995e00
Compare
daa5054
to
329250c
Compare
329250c
to
5ce0ebb
Compare
e995e00
to
f7e4bb7
Compare
This was referenced Aug 5, 2025
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.
Summary & Motivation
This creates a new StateBackedComponent class that takes advantage of the new StateStore object.
Subclasses of StateBackedComponent will implement two methods:
build_defs_from_state
: given a path to a state file, they will return a Definitions object. two things to note here:write_state_to_path
: the user code is given a path in which to write some state. The reason that the path is passed into this method is so that the framework can control the lifecycle of this path (i.e. making sure it gets cleaned up after we upload it).How I Tested These Changes
Changelog