Skip to content

[code intelligence] Update dim__orgs.sql #200

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

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

Conversation

ncarchedi
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 (8523553) code-intelligence (8936a5b)
Tables modified: 1 (details)
  • Different: 1
DEMO.CORE.DIM__ORGS
Primary keys ORG_ID
DIFFERENCES
 2 column(s) with differing values
column number of rows
SUB_PLAN ~404  58.2%
SUB_PRICE ~263  37.9%
View details →
 
6 potential data app dependencies
 
Unchanged Attributes
Total rows 694
Total columns 6
Schema changes 0
Common unique PKs 694
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.fct__yearly__financials (table) Run Data Diff →
DEMO.CORE.sales__sync (table) Run Data Diff →
DEMO.CORE.fct__monthly__financials (table) Run Data Diff →

@ncarchedi ncarchedi changed the title [test] Update dim__orgs.sql [code intelligence] Update dim__orgs.sql Jan 3, 2025
@dagadbm
Copy link

dagadbm commented Jan 10, 2025

✨ AI Overview

Here's an AI generated summary of your pull request that analyzes your code changes and their impact on your data. This feature is currently experimental, so please review the details carefully.

  1. The changes will recategorize subscription plans, marking all organizations with one or fewer users as 'Individual' regardless of their previous plan. This affects 404 organizations (58% of total) and will impact financial reporting in both Looker and Tableau dashboards.

  2. Null subscription prices will now default to 0 instead of remaining null, affecting 263 organizations (38% of total). This change will impact financial calculations in downstream tables like FCT__MONTHLY__FINANCIALS and FCT__YEARLY__FINANCIALS.

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