-
Notifications
You must be signed in to change notification settings - Fork 16
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
Debug postgresql #304
Debug postgresql #304
Conversation
…é dans db-fixtures uniquement dans ce cas là
…if path example file
…s de db fixtures - corrections bug test-dashboard et manage-corpus-user
Correction conflit debug postgresql
A faire: Test5: test recherche avec | (ne fonctionne pas car on recherche sein*, ce qui ne sort pas Seinz. |
Debug postgres conflit
This reverts commit feb226c.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I leave some comments for when you come back to this :)
@Juliettejns Il semble que le SET fasse planter un truc, d'une manière ou d'une autre. À mon avis, il faut transformer le warning en raise, et donc planter le serveur en cas de mauvaise locale. |
Il y a des plantages PSQL que je ne m'explique pas. |
d636c28
to
47f60ee
Compare
Small attempt to see if it fixes suff
47f60ee
to
05f8a75
Compare
J'ai essayé d'ajouter des Cf.
|
@Juliettejns Peux-tu tester voir si les crash psql ( |
|
Issues corrigées. |
c713906
to
0ee9db5
Compare
Branche de debug pour la version postgresql de pyrrha.
Log des erreurs avant debug:
Traitement de l'erreur 1 (test.registration_with_same_name) en rendant non obligatoire l'ajout d'une control List lors de l'ajout d'un corpus dans le test.
Traitement des erreurs 2 et 3 en autoincrémentant les control_list_id dans la fonction add_control_list située dans app.