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

Adding DISTINCT keyword #32

Merged
merged 1 commit into from
Sep 15, 2023
Merged

Adding DISTINCT keyword #32

merged 1 commit into from
Sep 15, 2023

Conversation

pgoslatara
Copy link
Owner

Description & motivation

To-do before merge

Screenshots:

Validation of models:

Changes to existing models:

Checklist:

  • My pull request represents one logical piece of work.
  • My commits are related to the pull request and look clean.
  • My SQL follows the dbt Labs style guide.
  • I have materialized my models appropriately.
  • I have added appropriate tests and documentation to any new models.
  • I have updated the README file if applicable.
  • I have added partitioning an clustering fields where appropriate.

@github-actions
Copy link

fct_bitcoin_blocks monitor

metrics cicd stg
row_cnt 1040 🔴 878 (-15.6%)
distinct_block_hash 1040 🔴 878 (-15.6%)
distinct_merkle_root 1040 🔴 878 (-15.6%)
distinct_created_at 1040 🔴 878 (-15.6%)
distinct_nonce 1040 🔴 878 (-15.6%)
distinct_bits 1 🟢 1 (0.0%)
sum_size 1705631633 🔴 1432703997 (-16.0%)
sum_stripped_size 813289619 🔴 688618841 (-15.3%)
sum_weight 4145500490 🔴 3498560520 (-15.6%)
sum_number 839473960 🔴 708638629 (-15.6%)
sum_version 675733274764 🔴 569691766888 (-15.7%)
sum_transaction_count 3580724 🔴 3004509 (-16.1%)

@github-actions
Copy link

dim_customers monitor

All values match!!
👍👍

@github-actions
Copy link

Outputs from manifest.json comparison:

Directly impacted models
stg_public_datasets__bitcoin_blocks
Indirectly impacted models
fct_bitcoin_blocks
stg_public_datasets__bitcoin_blocks

Impacted exposures:

Exposure type Name Owner
📊 DASHBOARD Tableau - fct_bitcoin_blocks Doug Douglas
📓 NOTEBOOK Databricks notebook - fct_bitcoin_blocks Frank Franklin
📱 APPLICATION Buy Bot - fct_bitcoin_blocks Chrissy Christie
🕵 ANALYSIS Hex - fct_bitcoin_blocks Bobby Bobson
🤖 ML MLflow - fct_bitcoin_blocks Emer Emerson

@pgoslatara pgoslatara merged commit 11e87aa into stg Sep 15, 2023
2 checks passed
@pgoslatara pgoslatara deleted the DE-23-distinct branch September 15, 2023 09:22
@github-actions
Copy link

The CD pipeline has started the backfill process...

@github-actions
Copy link

The CD pipeline has successfully finished the backfill process 🎉.

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