Skip to content

Conversation

leoebfolsom
Copy link
Collaborator

test change

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 (78491b3) leo (2047535)
Data Diffs of tables modified in this pull request: 1
  • Different: 1
DEMO.CORE.DIM_ORGS
Primary keys ORG_ID
DIFFERENCES
 Columns with changed type/order: 1
column change
NUM_USERS NUMBER(18,0) → NUMBER(1,0)  
 1 column(s) with differing values
column number of rows
NUM_USERS 56  35.7%
View details →
 
Unchanged Attributes
Total rows 157
Total columns 6
Common unique PKs 157
Added/removed rows 0
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