Skip to content

Commit

Permalink
remove unused code in materialize() (dagster-io#21628)
Browse files Browse the repository at this point in the history
unless I'm missing something
  • Loading branch information
johannkm authored and danielgafni committed Jun 18, 2024
1 parent 0204b2f commit 0b7db58
Showing 1 changed file with 0 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@

if TYPE_CHECKING:
from dagster._core.definitions.asset_selection import CoercibleToAssetSelection
from dagster._core.definitions.events import AssetKey

from ..execution.execute_in_process_result import ExecuteInProcessResult

Expand Down Expand Up @@ -90,11 +89,6 @@ def asset2(asset1):
partition_key = check.opt_str_param(partition_key, "partition_key")
resources = check.opt_mapping_param(resources, "resources", key_type=str)

all_executable_keys: Set[AssetKey] = set()
for asset in assets:
if isinstance(asset, AssetsDefinition):
all_executable_keys = all_executable_keys.union(set(asset.keys))

defs = Definitions(
jobs=[define_asset_job(name=EPHEMERAL_JOB_NAME, selection=selection)],
assets=assets,
Expand Down

0 comments on commit 0b7db58

Please sign in to comment.