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 #198

Open
wants to merge 4 commits into
base: master-databricks
Choose a base branch
from
Open

Conversation

leoebfolsom
Copy link
Collaborator

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.

View CI Run Details →

Base branch Pull Request branch
master-databricks (d62a8bf) databricks-patch-2 (d178453)
Tables modified: 2 (details)
  • Different: 2
demo.default.sales__sync
Primary keys org_id
master-databricks databricks-patch-2
DIFFERENCES
  Total rows 32 58 +81.2%
  Exclusive PKs 20 46
View details →
 
Unchanged Attributes
Total columns 2
Schema changes 0
Common unique PKs 12
Rows with NULL PKs 0
Rows with duplicate PKs 0
Columns with different values 0

demo.default.dim__orgs
Primary keys org_id
master-databricks databricks-patch-2
DIFFERENCES
  Total rows 494 804 +62.8%
  Exclusive PKs 0 310
 4 column(s) with differing values
column number of rows
created_at 494  100.0%
sub_plan 290  58.7%
sub_price 138  27.9%
num_users 15  3.0%
View details →
 
6 potential data app dependencies
 
Unchanged Attributes
Total columns 6
Schema changes 0
Common unique PKs 494
Rows with NULL PKs 0
Rows with duplicate PKs 0

Skipped Data Diffs of downstream tables: 2 Add "datafold:diff-all-downstream" label to this pull request to diff all affected tables
demo.default.fct__yearly__financials (table) Run Data Diff →
demo.default.fct__monthly__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-databricks (d62a8bf) databricks-patch-2 (0525cd3)
Tables modified: 2 (details)
  • Different: 2
demo.default.dim__orgs
Primary keys org_id
master-databricks databricks-patch-2
DIFFERENCES
  Total rows 494 804 +62.8%
  Exclusive PKs 0 310
 4 column(s) with differing values
column number of rows
created_at 494  100.0%
sub_plan 290  58.7%
sub_price 138  27.9%
num_users 15  3.0%
View details →
 
6 potential data app dependencies
 
Unchanged Attributes
Total columns 6
Schema changes 0
Common unique PKs 494
Rows with NULL PKs 0
Rows with duplicate PKs 0

demo.default.sales__sync
Primary keys org_id
master-databricks databricks-patch-2
DIFFERENCES
  Total rows 32 58 +81.2%
  Exclusive PKs 20 46
View details →
 
Unchanged Attributes
Total columns 2
Schema changes 0
Common unique PKs 12
Rows with NULL PKs 0
Rows with duplicate PKs 0
Columns with different values 0

Downstream tables: 2 (details)
  • Different: 2
demo.default.fct__yearly__financials
Primary keys date_year
DIFFERENCES
 2 column(s) with differing values
column number of rows
yearly_revenue 1  100.0%
yearly_subscribers 1  100.0%
View details →
 
3 potential data app dependencies
 
Unchanged Attributes
Total rows 1
Total columns 3
Schema changes 0
Common unique PKs 1
Added/removed rows 0
Rows with NULL PKs 0
Rows with duplicate PKs 0
 
 Modified upstream models   model.demo.dim__orgs

demo.default.fct__monthly__financials
Primary keys date_month
DIFFERENCES
 2 column(s) with differing values
column number of rows
sum_revenue 7  100.0%
cnt_subscribers 6  85.7%
View details →
 
6 potential data app dependencies
 
Unchanged Attributes
Total rows 7
Total columns 3
Schema changes 0
Common unique PKs 7
Added/removed rows 0
Rows with NULL PKs 0
Rows with duplicate PKs 0
 
 Modified upstream models   model.demo.dim__orgs

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.

2 participants