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

Update prisma #1031

Merged
merged 19 commits into from
Nov 8, 2021
Merged

Update prisma #1031

merged 19 commits into from
Nov 8, 2021

Conversation

Riron
Copy link
Collaborator

@Riron Riron commented Oct 12, 2021

MAJ Prisma - version 3.x qui permettait les gains de perf important, mais posait problème dans les tests d'inté.

Ils ont manifestement des fuites mémoires avec le nouvel API (cf prisma/prisma#8989): la connexion n'est jamais vraiment nettoyée car elle peut être réutilisée. Ca ne pose pas de problème en prod, mais comme jest recrée tout le contexte pour chaque test suite on a une fuite mémoire qui apparait. Ils réflechissent à ajouter une méthode à utiliser dans les tests type $dispose pour faire le ménage.

En attendant, on contourne le problème en splittant les tests dans plusieurs runners (4 pour le moment) => moins de tests, donc pas le temps de memory overload !
En bonus on gagne du temps sur le run. En malus, c'est un peu moins clair à lire dans les github actions.

Suite à la montée de version, Prisma a changé sa manière de gérer les champs nullable pour les champs Json. On ne peut plus utiliser null. A la place il faut utilise Prisma.DbNull ou Prisma.JsonNull. En lecture et en écriture. Ca complique les cas ou on faisait lecture du champ -> écriture, notamment pour la duplication de formulaires.
Pour "simplifier", j'ai passé les champs JSON en non nullable et valeur [] par défaut. Ce sont uniquement des packagingInfos, donc j'ai tendance à dire que la différence null | [] a peu d'importance - mais ça peut se discuter !

@Riron Riron force-pushed the mod/prisma-update branch 4 times, most recently from b7fb886 to 6335163 Compare October 19, 2021 07:51
Copy link
Member

@providenz providenz left a comment

Choose a reason for hiding this comment

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

Le front compile pas chez moi, après un generate ça passe.

@Riron Riron merged commit e7071f0 into dev Nov 8, 2021
@Riron Riron deleted the mod/prisma-update branch November 8, 2021 11:09
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.

None yet

3 participants