deprecate asset_partition_*_for_output on AssetExecutionContext

[jamiedemaria]

Summary & Motivation

Since we are wrapping up active work on the AssetExecutionContext for now, there are a couple of easy deprecations we'd like to get in that will improve user experience until we can more actively work on API development.

This PR deprecates the following methods in favor of:
asset_partition_key_for_output -> partition_key
asset_partition_keys_for_output -> partition_keys
asset_partitions_time_window_for_output -> partition_time_window
asset_partition_key_range_for_output -> partition_key_range

The majority of the changes in this PR are updating tests to use the non-deprecated APIs

How I Tested These Changes

updates unit tests

[jamiedemaria]

Current dependencies on/for this PR:

• master
  • PR Only have one kind of context for direct invocation #17554
    • PR duplicate op context methods on asset context  #18573
      • PR deprecate methods on AssetExecutionContext that can be found via the DagsterRun object #17339
        • PR Direct invocation asset execution context #18549
          • PR deprecate asset_partition_*_for_output on AssetExecutionContext #19436 👈
            • PR [docs] remove asset_partition_*_for_output from docs #19437
              • PR deprecate op related methods from AssetExecutionContext #19441
          • PR Make upstream asset materialization events available on the context #18971
            • PR [wip] store the tags fetched by data versioning so we can use them for upath io manager #19324
              • PR [UPath I/O managers] special case handling of None outputs #18820
            • PR Make input partitions methods based on asset key not input name #19027
              • PR Support partitioned assets for fetching latest AssetMaterializations #19286
              • PR [do not merge] base example to showcase new apis #19199
                • PR [prototype] sub-context for each upstream dependency  #19283
                • PR [prototype] AssetExecutionContext partition methods on sub context #19236
                • PR [prototype] AssetExecutionContext partition methods given as named tuple #19037
                • PR [prototype] AssetExecutionContext partition methods directly on the context #19032
            • PR [wip] two execution contexts with minimal changes #18543
              • PR [wip] see wht needs to change when asset context is no longer an op context #18553
        • PR support Asset and Op ExecutionContexts in standard execution path #17972
          • PR [wip] add AssetExecutionContext to direct invocation #18044

This stack of pull requests is managed by Graphite.

[sryza]

It looks like these get used frequently in docs, so we should make sure to fix those too before releasing this change.

[sryza]

We should also make sure that users don't hit deprecation warnings when they invoke non-deprecated methods that invoke these deprecated methods under the covers.

[jamiedemaria]

docs PR #19437

We should also make sure that users don't hit deprecation warnings when they invoke non-deprecated methods that invoke these deprecated methods under the covers.

Good point. I don't know of any methods calling these methods but I will do a search just to make sure Edit: I grepped through the code base and didn't find any instances where other methods called these methods. so we should be good on this front