-
Notifications
You must be signed in to change notification settings - Fork 18
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 dim_orgs.sql #138
base: master
Are you sure you want to change the base?
Update dim_orgs.sql #138
Conversation
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.
Use Datafold to diff your data, see the downstream impact, then post the results back to this PR. Add data diffs →
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.
Base branch | Pull Request branch |
master (37722f8) | ncarchedi-patch-6 (68d8923) |
- Different: 1
DEMO.CORE.DIM_ORGS | ||||||||||
Primary keys ORG_ID |
||||||||||
master | ncarchedi-patch-6 | |||||||||
DIFFERENCES | ||||||||||
Total rows | 157 | ↓ 50 -68.2% | ||||||||
Columns with changed type/order: 1
|
||||||||||
Exclusive PKs | 107 | 0 | ||||||||
1 column(s) with differing values
|
||||||||||
View details → | ||||||||||
11 potential data app dependencies | ||||||||||
Unchanged Attributes
|
Skipped Data Diffs of downstream tables: 3
Add "datafold:diff-all-downstream" label to this pull request to diff all affected tablesDEMO.CORE.sales_sync (table) | Run Data Diff → |
DEMO.CORE.fct_monthly_financials (table) | Run Data Diff → |
DEMO.CORE.fct_yearly_financials (table) | Run Data Diff → |
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.
Base branch | Pull Request branch |
master (37722f8) | ncarchedi-patch-6 (68d8923) |
- Identical: 3
DEMO.CORE.DIM_ORGS | ||||||||||||||||||
Primary keys ORG_ID |
||||||||||||||||||
View details → | ||||||||||||||||||
Unchanged Attributes
|
DEMO.CORE.DIM_USERS | ||||||||||||||||||
Primary keys USER_ID |
||||||||||||||||||
View details → | ||||||||||||||||||
Unchanged Attributes
|
DEMO.CORE.SALES_SYNC | ||||||||||||||||||
Primary keys ORG_ID |
||||||||||||||||||
View details → | ||||||||||||||||||
Unchanged Attributes
|
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.
Base branch | Pull Request branch |
master (78491b3) | ncarchedi-patch-6 (68d8923) |
- Different: 1
DEMO.CORE.DIM_ORGS | ||||||||||
Primary keys ORG_ID |
||||||||||
master | ncarchedi-patch-6 | |||||||||
DIFFERENCES | ||||||||||
Total rows | 157 | ↓ 50 -68.2% | ||||||||
Columns with changed type/order: 1
|
||||||||||
Exclusive PKs | 107 | 0 | ||||||||
1 column(s) with differing values
|
||||||||||
View details → | ||||||||||
11 potential data app dependencies | ||||||||||
Unchanged Attributes
|
Skipped Data Diffs of downstream tables: 3
Add "datafold:diff-all-downstream" label to this pull request to diff all affected tablesDEMO.CORE.sales_sync (table) | Run Data Diff → |
DEMO.CORE.fct_monthly_financials (table) | Run Data Diff → |
DEMO.CORE.fct_yearly_financials (table) | Run Data Diff → |
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.
Base branch | Pull Request branch |
master (78491b3) | ncarchedi-patch-6 (68d8923) |
- Different: 1
DEMO.CORE.DIM_ORGS | ||||||||||
Primary keys ORG_ID |
||||||||||
master | ncarchedi-patch-6 | |||||||||
DIFFERENCES | ||||||||||
Total rows | 157 | ↓ 50 -68.2% | ||||||||
Columns with changed type/order: 1
|
||||||||||
Exclusive PKs | 107 | 0 | ||||||||
1 column(s) with differing values
|
||||||||||
View details → | ||||||||||
11 potential data app dependencies | ||||||||||
Unchanged Attributes
|
Skipped Data Diffs of downstream tables: 3
Add "datafold:diff-all-downstream" label to this pull request to diff all affected tablesDEMO.CORE.sales_sync (table) | Run Data Diff → |
DEMO.CORE.fct_monthly_financials (table) | Run Data Diff → |
DEMO.CORE.fct_yearly_financials (table) | Run Data Diff → |
No description provided.