fix: remove duplicate partition_key defs, fix breaking_version to 2.0

- Remove the asset-partition-derived `partition_key` property that was
  silently overwriting the existing run-level `partition_key` property
  (`self._partition_key`) in both InputContext and OutputContext.
  The `asset_partition_key` → `partition_key` rename cannot be done
  until the run-level `partition_key` property is itself deprecated;
  that should be a separate PR.

- Remove the now-unreachable `@deprecated asset_partition_key` wrappers
  that delegated to the overwritten property.

- Fix `breaking_version="2.0.0"` → `"2.0"` across all new deprecation
  annotations to match every other deprecation in the execution/context
  package.

Addresses Greptile review feedback on PR #33896.

Author: sridhar-3009

python_modules/dagster/dagster/_core/execution/context/input.py

@@ -183,7 +183,7 @@ def config(self) -> Any:
         """The config attached to the input that we're loading."""
         return self._config
 
-    @deprecated(breaking_version="2.0.0", additional_warn_text="Use definition_metadata instead")
+    @deprecated(breaking_version="2.0", additional_warn_text="Use definition_metadata instead")
     @public
     @property
     def metadata(self) -> ArbitraryMetadataMapping | None:
@@ -336,35 +336,6 @@ def has_asset_partitions(self) -> bool:
         """Returns True if the asset being loaded as input is partitioned."""
         return self._asset_partitions_subset is not None
 
-    @public
-    @property
-    def partition_key(self) -> str:
-        """The partition key for input asset.
-
-        Raises an error if the input asset has no partitioning, or if the run covers a partition
-        range for the input asset.
-        """
-        subset = self._asset_partitions_subset
-
-        if subset is None:
-            check.failed("The input does not correspond to a partitioned asset.")
-
-        keys_iter = iter(subset.get_partition_keys())
-        first = next(keys_iter, None)
-        if first is not None and next(keys_iter, None) is None:
-            return first
-        check.failed(
-            f"Tried to access partition key for asset '{self.asset_key}', "
-            f"but the number of input partitions != 1: '{subset}'."
-        )
-
-    @deprecated(breaking_version="2.0.0", additional_warn_text="Use `partition_key` instead.")
-    @public
-    @property
-    def asset_partition_key(self) -> str:
-        """The partition key for input asset. Deprecated: use `partition_key`."""
-        return self.partition_key
-
     @public
     @property
     def partition_key_range(self) -> PartitionKeyRange:
@@ -391,7 +362,7 @@ def partition_key_range(self) -> PartitionKeyRange:
 
         return partition_key_ranges[0]
 
-    @deprecated(breaking_version="2.0.0", additional_warn_text="Use `partition_key_range` instead.")
+    @deprecated(breaking_version="2.0", additional_warn_text="Use `partition_key_range` instead.")
     @public
     @property
     def asset_partition_key_range(self) -> PartitionKeyRange:
@@ -412,7 +383,7 @@ def partition_keys(self) -> Sequence[str]:
 
         return list(self._asset_partitions_subset.get_partition_keys())
 
-    @deprecated(breaking_version="2.0.0", additional_warn_text="Use `partition_keys` instead.")
+    @deprecated(breaking_version="2.0", additional_warn_text="Use `partition_keys` instead.")
     @public
     @property
     def asset_partition_keys(self) -> Sequence[str]:
@@ -451,7 +422,7 @@ def partitions_time_window(self) -> TimeWindow:
 
         return time_window_for_partition_key_range(partitions_def, self.partition_key_range)
 
-    @deprecated(breaking_version="2.0.0", additional_warn_text="Use `partitions_time_window` instead.")
+    @deprecated(breaking_version="2.0", additional_warn_text="Use `partitions_time_window` instead.")
     @public
     @property
     def asset_partitions_time_window(self) -> TimeWindow:

python_modules/dagster/dagster/_core/execution/context/output.py

@@ -237,7 +237,7 @@ def run_id(self) -> str:
 
         return self._run_id
 
-    @deprecated(breaking_version="2.0.0", additional_warn_text="Use definition_metadata instead")
+    @deprecated(breaking_version="2.0", additional_warn_text="Use definition_metadata instead")
     @public
     @property
     def metadata(self) -> ArbitraryMetadataMapping | None:
@@ -464,42 +464,6 @@ def has_asset_partitions(self) -> bool:
 
         return self._asset_partitions_subset is not None
 
-    @public
-    @property
-    def partition_key(self) -> str:
-        """The partition key for output asset.
-
-        Raises an error if the output asset has no partitioning, or if the run covers a partition
-        range for the output asset.
-        """
-        if self._warn_on_step_context_use:
-            warnings.warn(
-                "You are using InputContext.upstream_output.partition_key. "
-                "This use on upstream_output is deprecated and will fail in the future. "
-                "Try to obtain what you need directly from InputContext. "
-                "For more details: https://github.com/dagster-io/dagster/issues/7900"
-            )
-
-        subset = self._asset_partitions_subset
-        if subset is None:
-            check.failed("The output does not correspond to a partitioned asset.")
-
-        keys_iter = iter(subset.get_partition_keys())
-        first = next(keys_iter, None)
-        if first is not None and next(keys_iter, None) is None:
-            return first
-        check.failed(
-            f"Tried to access partition key for asset '{self._asset_key}', "
-            f"but the number of output partitions != 1: '{subset}'."
-        )
-
-    @deprecated(breaking_version="2.0.0", additional_warn_text="Use `partition_key` instead.")
-    @public
-    @property
-    def asset_partition_key(self) -> str:
-        """The partition key for output asset. Deprecated: use `partition_key`."""
-        return self.partition_key
-
     @public
     @property
     def partition_key_range(self) -> PartitionKeyRange:
@@ -532,7 +496,7 @@ def partition_key_range(self) -> PartitionKeyRange:
 
         return partition_key_ranges[0]
 
-    @deprecated(breaking_version="2.0.0", additional_warn_text="Use `partition_key_range` instead.")
+    @deprecated(breaking_version="2.0", additional_warn_text="Use `partition_key_range` instead.")
     @public
     @property
     def asset_partition_key_range(self) -> PartitionKeyRange:
@@ -559,7 +523,7 @@ def partition_keys(self) -> Sequence[str]:
 
         return list(self._asset_partitions_subset.get_partition_keys())
 
-    @deprecated(breaking_version="2.0.0", additional_warn_text="Use `partition_keys` instead.")
+    @deprecated(breaking_version="2.0", additional_warn_text="Use `partition_keys` instead.")
     @public
     @property
     def asset_partition_keys(self) -> Sequence[str]:
@@ -604,7 +568,7 @@ def partitions_time_window(self) -> TimeWindow:
 
         return time_window_for_partition_key_range(partitions_def, self.partition_key_range)
 
-    @deprecated(breaking_version="2.0.0", additional_warn_text="Use `partitions_time_window` instead.")
+    @deprecated(breaking_version="2.0", additional_warn_text="Use `partitions_time_window` instead.")
     @public
     @property
     def asset_partitions_time_window(self) -> TimeWindow: