DatabaseCleaner improperly starts and ends transactions for ActiveRecord
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 38/100
Piste de recherche
Commencez par activerecord/lib/active_record/fixtures.rb, dans les sections référencées de début et de fin de transaction, puis comparez ce flux avec la manière dont ActiveRecord collecte les connexions disponibles. Vérifiez que la modification couvre chaque connexion du pool : les appels ultérieurs à ActiveRecord doivent rester transactionnels, et le nettoyage ne doit laisser aucune connexion avec une transaction active.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
If there is more than one connection in the connection pool DatabaseCleaner will only begin the transaction on one of the connections.
https://github.com/rails/rails/blob/master/activerecord/lib/active_record/fixtures.rb#L979
Any ActiveRecord call afterwards will use the next connection in the pool which does not have a transaction.
When DatabaseCleaner closes the transaction, it will only do so on the next active connection. Thus leaving a connection with an active transaction in the connection pool which can deadlock when doing a deletion cleanup or any table modifications.
DatabaseCleaner should start and end transactions the same way ActiveRecord does by collecting all available connections and starting transactions on all of them.
start: https://github.com/rails/rails/blob/06a9e18e646845ded0f0a46a14c1eb6b3a14e55a/activerecord/lib/active_record/fixtures.rb#L949
end: https://github.com/rails/rails/blob/06a9e18e646845ded0f0a46a14c1eb6b3a14e55a/activerecord/lib/active_record/fixtures.rb#L967
- Langage dominant
- Ruby
- Étoiles
- 76
- Forks
- 76
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Fournit un Dockerfile ou un fichier Docker Compose
- Aucun modèle de pull request
- Aucun guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de DatabaseCleaner/database_cleaner-active_record
-
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 30/100
DatabaseCleaner/database_cleaner-active_record#128 · 1 commentaire ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
DatabaseCleaner/database_cleaner-active_record#127 · 3 commentaires ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 45/100
-
Parallelized truncatesOuverte
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
Toutes les issues de DatabaseCleaner/database_cleaner-active_record
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 90/100
Les mainteneurs répondent en général sous 3 jours
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
jetrockets/jet_ui#47 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
FreeCAD/homebrew-freecad#870 ·
Les mainteneurs répondent en général sous 1 jour