Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CT-2521] Define SemanticModel node for the Manifest #7499

Closed
Tracked by #7498
QMalcolm opened this issue May 3, 2023 · 0 comments · Fixed by #7769
Closed
Tracked by #7498

[CT-2521] Define SemanticModel node for the Manifest #7499

QMalcolm opened this issue May 3, 2023 · 0 comments · Fixed by #7769
Assignees

Comments

@QMalcolm
Copy link
Contributor

QMalcolm commented May 3, 2023

There are two new node types being added to the manifest as part of the MetricFlow integration work: SemanticModel nodes and MetricNodes. This ticket addresses the SemanticModel nodes. In dbt-semantic-interfaces we're adding a protocol of a SemanticModel, which will have the shared standard definition of a SemanticModel. In dbt-core we need to compose this protocol with base node classes to define dbt-core's representation of a SemanticModel. Additionally there are some additional fields we should expose as defined in #7456

Blocked by:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants