-
Notifications
You must be signed in to change notification settings - Fork 944
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
combine and simplify sq and exports #5470
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@@ -36,19 +84,19 @@ saved_queries: | |||
cache: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add file name semantic model
|
||
## Configure saved query | ||
|
||
Use saved queries to define and manage common Semantic Layer queries in YAML, including metrics and dimensions. Saved queries enable you to organize and reuse common MetricFlow queries within dbt projects. For example, you can group related metrics together for better organization, and include commonly used dimensions and filters. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Include dbt_project.yml config (saved-queries) blurb
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Left a few comments.
| `description` | String | Required | A description of the saved query. | | ||
| `label` | String | Required | The display name for your saved query. This value will be shown in downstream tools. | | ||
| `config` | String | Required | A config section for any parameters specifying the saved query. | | ||
| `config::cache` | String | Optional | A boolean to specify if a saved query should be used to populate the cache. Accepts `true` or `false`. Defaults to `false` | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this really required or optional? m
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Jordan confirmed it's optional
this pr consolidates the saved queries and exports docs so its clearer how both work together. it also removes the comparison table and adds a 'configure exports' section in teh saved queries page, making sure they flow correctly.
notion project