[Hactoberfest] Limit the number of retries on concurrency modification exceptions
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 35/100
Piste de recherche
Commencez dans src/main/java/com/commercetools/sync/commons/BaseSync.java aux lignes 115-124, où les exceptions de modification concurrente déclenchent actuellement des nouvelles tentatives de synchronisation. Suivez ce chemin de nouvelle tentative et déterminez comment la limite de nouvelles tentatives doit être appliquée ; le travail est terminé lorsque les requêtes cessent d’être relancées après la limite configurée au lieu de continuer jusqu’à leur réussite.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Situation
Currently on concurrency modification exceptions the sync retries with no limit until there is no error. See:
https://github.com/commercetools/commercetools-sync-java/blob/7cacc37ef8a1412f9bf300ba08000c108db81b4f/src/main/java/com/commercetools/sync/commons/BaseSync.java#L115-L124
Complication
It causes unnecessary api calls if the platform returns 409 - ConcurrentModificationException for the requests.
Resolution
Limit the number of retries on concurrency modification exceptions.
- Langage dominant
- Java
- Étoiles
- 41
- Forks
- 41
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le 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 commercetools/commercetools-sync-java
-
Dependency DashboardOuverte
Difficulté 4/5 3-5 jours Accessibilité débutants 20/100
-
Difficulté 3/5 1-2 jours Accessibilité débutants 52/100
-
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
-
Difficulté 4/5 3-5 jours Accessibilité débutants 32/100
commercetools/commercetools-sync-java#1201 · 4 commentaires ·
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 20/100
Toutes les issues de commercetools/commercetools-sync-java
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
apache/arrow-java#1311 ·
Les mainteneurs répondent en général sous 2 jours
-
bug triage
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
Les mainteneurs répondent en général sous 1 jour
-
security
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
IBM/networking-java-sdk#204 ·