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

feat(pipeline) : Setup deduplication as a new processings module #350

Merged
merged 1 commit into from
Jan 13, 2025

Conversation

vperron
Copy link
Contributor

@vperron vperron commented Dec 10, 2024

No description provided.

@vperron vperron self-assigned this Dec 10, 2024
@vperron vperron force-pushed the vperron/processings-dedupe branch 3 times, most recently from dbbc875 to 72e8608 Compare December 11, 2024 11:27
@vperron vperron added the deploy-to-staging Permet d'activer le déploiement de la PR en staging. label Dec 11, 2024
@vperron vperron force-pushed the vperron/processings-dedupe branch from 72e8608 to 60764e6 Compare December 11, 2024 14:23
@vperron vperron force-pushed the vperron/processings-dedupe branch from 60764e6 to 4392ace Compare December 11, 2024 14:46
@hlecuyer hlecuyer force-pushed the vperron/processings-dedupe branch from 4392ace to 287ea1f Compare December 17, 2024 10:27
@hlecuyer
Copy link
Contributor

J'ai rebase pour deployer en staging l'import des tables analytics en staging.

@vmttn vmttn removed the deploy-to-staging Permet d'activer le déploiement de la PR en staging. label Dec 18, 2024
@vperron vperron force-pushed the vperron/processings-dedupe branch from 0a1b5ab to 23f210d Compare January 6, 2025 09:30
@vperron vperron added the deploy-to-staging Permet d'activer le déploiement de la PR en staging. label Jan 6, 2025
@vperron vperron marked this pull request as ready for review January 6, 2025 09:41
@vperron vperron force-pushed the vperron/processings-dedupe branch from 23f210d to 12a2209 Compare January 6, 2025 15:31
@vperron vperron force-pushed the vperron/processings-dedupe branch from 12a2209 to e72410c Compare January 6, 2025 16:03
@vperron vperron removed the deploy-to-staging Permet d'activer le déploiement de la PR en staging. label Jan 7, 2025
@vperron vperron force-pushed the vperron/processings-dedupe branch from e72410c to cc48622 Compare January 9, 2025 14:43
Copy link
Contributor

@vmttn vmttn left a comment

Choose a reason for hiding this comment

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

à part un petit point, nickel 💯 💯 💯

l'intuition que int__doublons_nb_cross_source et int__doublons_nb_mono_source pourraient être un seul et même modèle, mais ça ira très bien comme ça 👍

(.venv) [victor@W]<~/dev/data-inclusion/pipeline/dbt> time dbt run -s models/intermediate/int__doublons_structures.sql
10:06:25  Running with dbt=1.8.6
10:06:25  Registered adapter: postgres=1.8.2
10:06:25  Unable to do partial parsing because of a version mismatch
10:06:28  Found 137 models, 12 snapshots, 10 seeds, 793 data tests, 52 sources, 554 macros, 5 unit tests
10:06:28
10:06:28  Concurrency: 1 threads (target='dev')
10:06:28
10:06:28  1 of 1 START sql table model public_intermediate.int__doublons_structures ...... [RUN]
10:06:49  1 of 1 OK created sql table model public_intermediate.int__doublons_structures . [SELECT 9457 in 20.66s]
10:06:49
10:06:49  Finished running 1 table model in 0 hours 0 minutes and 20.93 seconds (20.93s).
10:06:49
10:06:49  Completed successfully
10:06:49
10:06:49  Done. PASS=1 WARN=0 ERROR=0 SKIP=0 TOTAL=1

real    0m25,923s
user    0m4,834s
sys     0m0,077s
@vperron vperron force-pushed the vperron/processings-dedupe branch from cc48622 to 91a610b Compare January 13, 2025 09:05
@vperron vperron merged commit 3b7fbc7 into main Jan 13, 2025
6 checks passed
@vperron vperron deleted the vperron/processings-dedupe branch January 13, 2025 09:45
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.

3 participants