Skip to content

Commit

Permalink
Merge branch 'current' into ENTERPRISE-84-end-user-saml-sso-using-goo…
Browse files Browse the repository at this point in the history
…gle-idp-should-get-group-membership
  • Loading branch information
robotmiller authored Sep 25, 2023
2 parents b0ff48c + 2682f2b commit 6b79f30
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ id: 2-how-we-style-our-sql

- 🔙 Fields should be stated before aggregates and window functions.
- 🤏🏻 Aggregations should be executed as early as possible (on the smallest data set possible) before joining to another table to improve performance.
- 🔢 Ordering and grouping by a number (eg. group by 1, 2) is preferred over listing the column names (see [this classic rant](https://blog.getdbt.com/write-better-sql-a-defense-of-group-by-1/) for why). Note that if you are grouping by more than a few columns, it may be worth revisiting your model design.
- 🔢 Ordering and grouping by a number (eg. group by 1, 2) is preferred over listing the column names (see [this classic rant](https://www.getdbt.com/blog/write-better-sql-a-defense-of-group-by-1) for why). Note that if you are grouping by more than a few columns, it may be worth revisiting your model design.

## Joins

Expand Down
4 changes: 2 additions & 2 deletions website/docs/reference/dbt-jinja-functions/builtins.md
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
---
title: "About builtins Jinja function"
title: "About builtins Jinja variable"
sidebar_label: "builtins"
id: "builtins"
description: "Read this guide to understand the builtins Jinja function in dbt."
description: "Read this guide to understand the builtins Jinja variable in dbt."
---


Expand Down

0 comments on commit 6b79f30

Please sign in to comment.