dagster icon indicating copy to clipboard operation
dagster copied to clipboard

[DS][24/n] Remove all AssetSubset references from the context and result objects

Open OwenKephart opened this issue 9 months ago • 1 comments

Summary & Motivation

As title -- for the new codepaths, helps avoid a lot of convert_to_valid_asset_subset() business.

I didn't bother updating the AutoMaterializeRule side of the world in a nicer way, as this code will eventually be removed anyway.

Had to add some new methods to replace the & / | / - operators, this time properly labeled with "compute_"

How I Tested These Changes

OwenKephart avatar May 02 '24 22:05 OwenKephart

  • #21641 Graphite: 2 dependent PRs (#21670 Graphite, #21671 Graphite)
  • #21640 Graphite
  • #21648 Graphite
  • #21615 Graphite
  • #21613 Graphite
  • #21612 Graphite 👈
  • #21573 Graphite
  • #21546 Graphite
  • #21545 Graphite
  • #21541 Graphite
  • #21540 Graphite
  • #21539 Graphite
  • #21538 Graphite
  • #21537 Graphite
  • #21536 Graphite
  • #21535 Graphite
  • #21521 Graphite
  • #21520 Graphite
  • #21511 Graphite: 1 other dependent PR (#21512 Graphite)
  • #21510 Graphite
  • #21508 Graphite
  • #21507 Graphite
  • #21505 Graphite
  • #21504 Graphite
  • #21503 Graphite
  • #21502 Graphite
  • #21501 Graphite
  • #21500 Graphite
  • #21499 Graphite
  • master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @OwenKephart and the rest of your teammates on Graphite Graphite

OwenKephart avatar May 02 '24 22:05 OwenKephart

Merge activity

  • May 3, 5:31 PM EDT: @OwenKephart started a stack merge that includes this pull request via Graphite.
  • May 3, 6:25 PM EDT: Graphite rebased this pull request as part of a merge.
  • May 3, 6:28 PM EDT: @OwenKephart merged this pull request with Graphite.

OwenKephart avatar May 03 '24 21:05 OwenKephart