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 dim_orgs.sql #138

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update dim_orgs.sql #138

wants to merge 1 commit into from

Conversation

ncarchedi
Copy link
Contributor

No description provided.

Copy link

@datafold datafold bot left a 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 →

Copy link

@datafold datafold bot left a comment

Choose a reason for hiding this comment

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

View CI Run Details →

Base branch Pull Request branch
master (37722f8) ncarchedi-patch-6 (68d8923)
Data Diffs of tables modified in this pull request: 1
  • 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
column change
SUB_PRICE NUMBER(38,0) → NUMBER(1,0)  
  Exclusive PKs 107 0
 1 column(s) with differing values
column number of rows
SUB_PRICE 19  38.0%
View details →
 
11 potential data app dependencies
 
Unchanged Attributes
Total columns 6
Common unique PKs 50
Rows with NULL PKs 0
Rows with duplicate PKs 0

Skipped Data Diffs of downstream tables: 3 Add "datafold:diff-all-downstream" label to this pull request to diff all affected tables
DEMO.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 →

Copy link

@datafold datafold bot left a comment

Choose a reason for hiding this comment

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

View CI Run Details →

Base branch Pull Request branch
master (37722f8) ncarchedi-patch-6 (68d8923)
Data Diffs of tables modified in this pull request: 3
  • Identical: 3
DEMO.CORE.DIM_ORGS
Primary keys ORG_ID
View details →
 
Unchanged Attributes
Total rows 157
Total columns 6
Schema changes 0
Common unique PKs 157
Added/removed rows 0
Rows with NULL PKs 0
Rows with duplicate PKs 0
Columns with different values 0

DEMO.CORE.DIM_USERS
Primary keys USER_ID
View details →
 
Unchanged Attributes
Total rows 475
Total columns 1
Schema changes 0
Common unique PKs 475
Added/removed rows 0
Rows with NULL PKs 0
Rows with duplicate PKs 0
Columns with different values 0

DEMO.CORE.SALES_SYNC
Primary keys ORG_ID
View details →
 
Unchanged Attributes
Total rows 4
Total columns 2
Schema changes 0
Common unique PKs 4
Added/removed rows 0
Rows with NULL PKs 0
Rows with duplicate PKs 0
Columns with different values 0

@ncarchedi ncarchedi closed this Jul 9, 2024
@ncarchedi ncarchedi reopened this Jul 9, 2024
Copy link

@datafold datafold bot left a comment

Choose a reason for hiding this comment

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

View CI Run Details →

Base branch Pull Request branch
master (78491b3) ncarchedi-patch-6 (68d8923)
Data Diffs of tables modified in this pull request: 1
  • 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
column change
SUB_PRICE NUMBER(38,0) → NUMBER(1,0)  
  Exclusive PKs 107 0
 1 column(s) with differing values
column number of rows
SUB_PRICE 19  38.0%
View details →
 
11 potential data app dependencies
 
Unchanged Attributes
Total columns 6
Common unique PKs 50
Rows with NULL PKs 0
Rows with duplicate PKs 0

Skipped Data Diffs of downstream tables: 3 Add "datafold:diff-all-downstream" label to this pull request to diff all affected tables
DEMO.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 →

Copy link

@datafold datafold bot left a comment

Choose a reason for hiding this comment

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

View CI Run Details →

Base branch Pull Request branch
master (78491b3) ncarchedi-patch-6 (68d8923)
Data Diffs of tables modified in this pull request: 1
  • 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
column change
SUB_PRICE NUMBER(38,0) → NUMBER(1,0)  
  Exclusive PKs 107 0
 1 column(s) with differing values
column number of rows
SUB_PRICE 19  38.0%
View details →
 
11 potential data app dependencies
 
Unchanged Attributes
Total columns 6
Common unique PKs 50
Rows with NULL PKs 0
Rows with duplicate PKs 0

Skipped Data Diffs of downstream tables: 3 Add "datafold:diff-all-downstream" label to this pull request to diff all affected tables
DEMO.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 →

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant