You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As part of an effort to move any ArtifactMixin subclasses into the top-level dbt/artifacts directory, we need to move the Exposure contract (just data types), which WritableManifest depends on.
Acceptance criteria
Exposure (and its necessary dependencies) are moved to a new dbt/artifacts directory. dbt/artifacts should have no additional dependencies on dbt.core except for from contracts.nodes which are actively being refactored:
GraphNode can also be split into data (in dbt/artifacts) and functional classes
ExposureConfig depends on common BaseConfig and just adds data fields so should be straightforward to move as well.
Exposure should be split into data & functional interfaces, defined in dbt/artifacts in dbt/core respectively. Additionally, any inheritance hierarchy between concrete node classes should be flattened as part of this spike.
Housekeeping
Short description
As part of an effort to move any
ArtifactMixin
subclasses into the top-leveldbt/artifacts
directory, we need to move theExposure
contract (just data types), which WritableManifest depends on.Acceptance criteria
Exposure (and its necessary dependencies) are moved to a new dbt/artifacts directory. dbt/artifacts should have no additional dependencies on dbt.core except for from contracts.nodes which are actively being refactored:
GraphNode
can also be split into data (in dbt/artifacts) and functional classesExposureConfig
depends on commonBaseConfig
and just adds data fields so should be straightforward to move as well.Exposure should be split into data & functional interfaces, defined in dbt/artifacts in dbt/core respectively. Additionally, any inheritance hierarchy between concrete node classes should be flattened as part of this spike.
Should follow initial pattern set in #9379
Impact to Other Teams
N/A - should be a refactor.
Will backports be required?
nope
Context
No response
The text was updated successfully, but these errors were encountered: