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

Update metricflow-commands.md #6181

Merged
merged 8 commits into from
Sep 30, 2024
Merged

Update metricflow-commands.md #6181

merged 8 commits into from
Sep 30, 2024

Conversation

mirnawong1
Copy link
Contributor

updating query examples to include missing primary entity prefix in group by command

refer to slack thread

updating query examples to include missing primary entity prefix in group by command

[refer to slack thread](https://dbt-labs.slack.com/archives/C04REFN03MX/p1727728803420059)
@mirnawong1 mirnawong1 requested a review from a team as a code owner September 30, 2024 21:09
Copy link

vercel bot commented Sep 30, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
docs-getdbt-com ✅ Ready (Inspect) Visit Preview Sep 30, 2024 9:33pm

@github-actions github-actions bot added content Improvements or additions to content size: small This change will take 1 to 2 days to address Docs team Authored by the Docs team @dbt Labs labels Sep 30, 2024
Copy link
Contributor

@WilliamDee WilliamDee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks!


# In dbt Core
mf query --metrics order_total --group-by metric_time,is_food_order --limit 10 --order-by -metric_time
mf query --metrics order_total --group-byprimary_entity__is_food_order --limit 10 --order-by -metric_time
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

missing a space between --group-by


**Query**
```bash
dbt sl query --metrics order_total --group-by metric_time,is_food_order # In dbt Cloud
dbt sl query --metrics order_total --group-by primary_entity__is_food_order # In dbt Cloud
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's use order_id__is_food_order instead of primary_entity__is_food_order since that's what is used in teh where clause tab

@mirnawong1
Copy link
Contributor Author

thank you @WilliamDee ! folded these in -- appreciate the quick review!!

website/docs/docs/build/metricflow-commands.md Outdated Show resolved Hide resolved
website/docs/docs/build/metricflow-commands.md Outdated Show resolved Hide resolved
website/docs/docs/build/metricflow-commands.md Outdated Show resolved Hide resolved
website/docs/docs/build/metricflow-commands.md Outdated Show resolved Hide resolved
@mirnawong1 mirnawong1 merged commit 22802ec into current Sep 30, 2024
10 checks passed
@mirnawong1 mirnawong1 deleted the mirnawong1-patch-26 branch September 30, 2024 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content Improvements or additions to content Docs team Authored by the Docs team @dbt Labs size: small This change will take 1 to 2 days to address
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants