commercetools / commercetools/commercetools-sync-java

[Hactoberfest] Limit the number of retries on concurrency modification exceptions

Aperta
#197 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Hacktoberfest
Lingua principale
Java
Stelle
40
Fork
41
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

**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.

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia in src/main/java/com/commercetools/sync/commons/BaseSync.java alle righe 115-124, dove le eccezioni di modifica concorrente attivano attualmente nuovi tentativi di sincronizzazione. Segui quel percorso di tentativi e determina come debba essere applicato il limite dei tentativi; il lavoro è completato quando le richieste smettono di essere ritentate dopo il limite configurato invece di continuare fino al successo.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
java
Ambito
backend
Tipo di issue
Bug
Difficoltà
2/5
Tempo stimato
1-3 ore
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.