search

dataforgoodfr / 12_bloom

23 stars 11 forks source link

Revoir la mapping SQLAchemy ou étudier une migration vers SQLModel #128

Open njouanin opened 4 months ago

njouanin commented 4 months ago

Le code actuel utilise la syntaxe impérative de SQLAlchemy. Depuis SQLAlchemy 2, la syntaxe déclarative est préconisée => modifier le mapping (src/bloom/infra/database/sql_model.py) pour utiliser cette syntaxe.

OU: étudier la migration vers SQLModel, qui permet de fusionner le mapping Pydantic avec le mapping SQLAlchemy.

njouanin commented 3 months ago

note: SQLModel ne parait pas très stable pour l'instant.

rv2931 commented 3 months ago

note: SQLModel ne parait pas très stable pour l'instant.

Ok... Bin c'était une idée mais pour avoir mis le nez dans la partie infra database je vois complètement l'intérêt de ce SqlModel parce qu'en l'état c'est bien lourd je touvrpour avoir et les avantages de Pydantic et l'ORM alchemy... Mais soit si tu penses que c'est pas une bonne idée en l'état faut abandonner l'idée. Toutes façon très clairement on aura pas le temps là

njouanin commented 3 months ago

C'est une remarque qu'avait fait quelqu'un sur Slack.

rv2931 commented 2 months ago

J'ai regardé un peu. FastAPI est quand même une référence aujourd'hui à priori et le lien avec SQLModel qui est maintenu par la même équipe semble assez courant La migration semble assez facile à priori vu que les SQLModel héritent des BaseModel de pydantic https://www.youtube.com/watch?v=pRYzMF04fLw