Skip to content

Adding rule to eliminate DAX Calculated Columns #253

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.

Sign up for GitHub

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

Open
KratosDataGod wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Adding rule to eliminate DAX Calculated Columns #253

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
613cb66
Adding rule to eliminate DAX Calculated Columns
KratosDataGod Mar 19, 2024
087d520
Upping severity to 3 to fail code deployments - NO DAX COLUMNS
KratosDataGod Mar 19, 2024
4266366
Changing Severity to 2 per BPA rules
KratosDataGod Mar 20, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 10 additions & 0 deletions BestPracticeRules/BPARules.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -220,6 +220,16 @@
"Expression": "AllColumns.Where(Type.ToString() == \"Calculated\").Count() > 5",
"CompatibilityLevel": 1200
},
{
"ID": "NO_CALCULATED_COLUMNS",
"Name": "[Performance] No DAX Columns - Eliminate the use of calculated columns",
"Category": "Performance",
"Description": "Eliminate the use of Calcualted Columns. In addition to SQLBI's guidance that Calculated columns: do not compress as well as data columns so they take up more memory, slow down processing times for both the table as well as process recalc. Offload calculated column logic to your data warehouse and turn these calculated columns into data columns. 16 additional reasons to eliminate DAX Calculated Columns - \r\nReference: https://www.youtube.com/watch?v=oKuO0_17Y5o",
"Severity": 2,
"Scope": "Model",
"Expression": "AllColumns.Where(Type.ToString() == \"Calculated\").Count() > 0",
"CompatibilityLevel": 1200
},
{
"ID": "CHECK_IF_BI-DIRECTIONAL_AND_MANY-TO-MANY_RELATIONSHIPS_ARE_VALID",
"Name": "[Performance] Check if bi-directional and many-to-many relationships are valid",
Expand Down