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

refacto: begin refacto uniformisation repositories #218

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

rv2931
Copy link
Collaborator

@rv2931 rv2931 commented Oct 27, 2024

L'idée est refactoriser et uniformiser les repositories

  • utilisation du repository pattern complet avec repository génériques/templates
  • Les repositories prennent des calsses Domain en entrées et renvoient des classes Domain en retour
  • utilisation des mappers Model/Domain et Domain/model à l'intérieur des repositories
  • pour les fonctions repos qui renvoient aujourd'hui des dataframe ou autres types non Domain, créer des mappers adéquats Domain/Dataframe

Réflexions:
Actuellement les repositories prennent les session_factory en paramètre de constructeur cependant il n'est jamais utilisé afin de pouvoir partager une session avec plusieurs repositories et utiliser les fonctions session.begin()/Rollback() sur tout un pipeline
je me pose donc la question de garder cette interdépendance session_factory et potentiellement garder la solution actuelle à savoir fournir la session directement en paramètre des fonctions de repository

@rv2931 rv2931 marked this pull request as draft October 27, 2024 20:12
@rv2931 rv2931 force-pushed the refacto_repositories branch 2 times, most recently from d3def35 to c2d5007 Compare November 10, 2024 23:49
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