Skip to content

Commit

Permalink
add kwargs to pipeline snapshot from storage (#8149)
Browse files Browse the repository at this point in the history
  • Loading branch information
jamiedemaria committed Jun 1, 2022
1 parent 0e6f9a9 commit 64bd480
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions python_modules/dagster/dagster/core/snap/pipeline_snapshot.py
Original file line number Diff line number Diff line change
Expand Up @@ -90,6 +90,7 @@ def _pipeline_snapshot_from_storage(
lineage_snapshot: Optional["PipelineSnapshotLineage"] = None,
graph_def_name: Optional[str] = None,
metadata: Optional[List[Union[MetadataEntry, PartitionMetadataEntry]]] = None,
**kwargs, # pylint: disable=unused-argument
) -> "PipelineSnapshot":
"""
v0
Expand All @@ -99,6 +100,10 @@ def _pipeline_snapshot_from_storage(
- graph_def_name
v3:
- metadata added
v4:
- add kwargs so that if future versions add new args, this version of deserialization will
be able to ignore them. previously, new args would be passed to old versions and cause
deserialization errors
"""
if graph_def_name is None:
graph_def_name = name
Expand Down

0 comments on commit 64bd480

Please sign in to comment.