HomePhabricator

Configured refactor: IConfigMappable --> ConfiguredMixin

Description

Configured refactor: IConfigMappable --> ConfiguredMixin

Summary:
I was scoping out configured APIs and got frustrated at the
code and documentation duplication. This is my attempt to reconcile
this.

There is definitely some oddness to componesate for the fact
that the node variants (graph and solid) can inherit their
name from the decorated function and other variants cannot. That
causes the passing of original_config_or_config_fn to
copy_for_configured.

However I still like the reduction of surface area. I would like to
reconcile configurable and config mapping into a single capability at
some point, then they both perform the same core task to some degree.

Test Plan: BK

Reviewers: sashank, alangenfeld, sandyryza

Reviewed By: sandyryza

Differential Revision: https://dagster.phacility.com/D5192

Details

Provenance
schrocknAuthored on Wed, Nov 18, 8:09 PM
Reviewer
sandyryza
Differential Revision
D5192: Configured refactor: IConfigMappable --> ConfiguredMixin
Parents
R1:32e341604fbb: Use yield from in dagster.core
Branches
Unknown
Tags
Unknown