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

👁️‍🗨️ Vue matérialisée pour tous les acteurs #1063

Merged
merged 6 commits into from
Nov 26, 2024

Conversation

maxcorbeau
Copy link
Contributor

👁️‍🗨️ Vue matérialisée pour tous les acteurs

Carte Notion

  • QUOI: un DAG pour créer/gérer la vue matérialisée qfdmo_vue_acteur_tous combinant les acteurs des tables qfdmo_displayedacteur, qfdmo_revisionacteur, qfdmo_acteur
  • POURQUOI: solutionner les problèmes suivants:
    • Besoin de construire des requêtes SQL complexes pour avoir une vue d’ensemble
    • Risque d’erreur dans la construction des requêtes
    • Duplication des efforts pour @chrischarousset et moi-même
  • COMMENT:
    • un nouveau dossier /dags/views/ pour gérer les vues (peut être un jour on bascule sur un framework dédié comme dbt)
    • un DAG tout simple qui drop/créer la vue
    • le SQL de la vue est généré en Python

Validation

  • La vue fonctionne sur PREPROD et semble donner des résultats satisfaisants
  • J'ai demandé à @chrischarousset de valider d'un point de vue métier (et si il veut étendre la vue avec d'autres champs)
  • @kolok : si tu peux valider la logique de base de la vue STP

Améliorations futures

  • Utiliser Django models ou SQL Alchemy pour générer la value (semble overkill pour l'instant)

@maxcorbeau maxcorbeau requested a review from a team as a code owner November 25, 2024 14:23
@maxcorbeau maxcorbeau requested review from kolok, fabienheureux and chrischarousset and removed request for a team and fabienheureux November 25, 2024 14:23
Copy link
Member

@fabienheureux fabienheureux left a comment

Choose a reason for hiding this comment

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

J'ai fait 2-3 commentaires sur la forme (je passais par là...), mais c'est vraiment histoire de commenter.

C'est globalement très clair 👍

Copy link
Contributor

@kolok kolok left a comment

Choose a reason for hiding this comment

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

Hello @maxcorbeau

La question que je me pose est : quelle est l'utilisation que l'on veut faire de cette vue qui n'est pas possible ou disponible avec la table displayedacteur ?

Dispo pour en parler ensemble demain :)

@chrischarousset
Copy link
Contributor

Hello @kolok , j'ai mis un exemple dans la carte : trouver tous les acteurs qui sont des acteurs-chapeau.

@maxcorbeau
Copy link
Contributor Author

Hello @maxcorbeau

La question que je me pose est : quelle est l'utilisation que l'on veut faire de cette vue qui n'est pas possible ou disponible avec la table displayedacteur ?

Dispo pour en parler ensemble demain :)

@nico: hello, voir 🎮 Exemples de requête

dags/views/view_acteur_tous.py Outdated Show resolved Hide resolved
dags/views/sql/view_acteur_tous.sql Show resolved Hide resolved
Copy link
Contributor

@kolok kolok left a comment

Choose a reason for hiding this comment

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

Un mini-typo mais c'est ok pour moi

dags/views/sql/generators/README.md Outdated Show resolved Hide resolved
@kolok kolok force-pushed the vue_acteur_tous_v1 branch from 578c19e to 74c477b Compare November 26, 2024 16:16
@kolok kolok enabled auto-merge (squash) November 26, 2024 16:17
@kolok kolok merged commit 06e065f into main Nov 26, 2024
7 checks passed
@kolok kolok deleted the vue_acteur_tous_v1 branch November 26, 2024 16:25
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.

4 participants