From 4543a2ed693cd32484e83bb144a80e7fcae5f154 Mon Sep 17 00:00:00 2001 From: JamieDeMaria Date: Wed, 27 Sep 2023 12:56:19 -0400 Subject: [PATCH] update comment --- .../dagster/_core/definitions/decorators/asset_decorator.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/python_modules/dagster/dagster/_core/definitions/decorators/asset_decorator.py b/python_modules/dagster/dagster/_core/definitions/decorators/asset_decorator.py index 548ee4cb19148..6dca815d874f8 100644 --- a/python_modules/dagster/dagster/_core/definitions/decorators/asset_decorator.py +++ b/python_modules/dagster/dagster/_core/definitions/decorators/asset_decorator.py @@ -1322,7 +1322,8 @@ def _make_asset_deps(deps: Optional[Iterable[CoercibleToAssetDep]]) -> Optional[ # we cannot do deduplication via a set because MultiPartitionMappings have an internal # dictionary that cannot be hashed. Instead deduplicate by making a dictionary and checking - # for existing keys. + # for existing keys. If an asset is specified as a dependency more than once, only error if the + # dependency is different (ie has a different PartitionMapping) if ( asset_dep.asset_key in dep_dict.keys() and asset_dep != dep_dict[asset_dep.asset_key]