milvus-io/milvus

[Feature]: Add retry for etcd operation

Geschlossen

#17.569 geöffnet am 15.06.2022

Auf GitHub ansehenContributor Guide
 (3 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Go (4.000 Forks)batch import
good first issuekind/feature

Repository-Metriken

Stars
 (44.298 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 7T) (314 gemergte PRs in 30 T)

Beschreibung

Is there an existing issue for this?

  • I have searched the existing issues

Is your feature request related to a problem? Please describe.

Milvus highly rely on etcd for meta operation. Retry is need since failed to operate on etcd may cause server crash. To be noticed, the CompareValueAndSwap, CompareVersionAndSwap may no be able to add retry in a no brainer style so we need to take care about it.

Describe the solution you'd like.

No response

Describe an alternate solution.

No response

Anything else? (Additional Context)

No response

Contributor Guide

Research-Richtung
Untersuche die bestehende etcd Nutzung in Milvus, insbesondere die Funktionen CompareValueAndSwap und CompareVersionAndSwap. Sieh dir an, wie andere Teile von Milvus Wiederholungsversuche handhaben. Schlage einen Wiederholungsmechanismus mit exponentiellem Backoff vor und berücksichtige Idempotenz für CAS Operationen. Überprüfe die Kommentare auf frühere Diskussionen.
Tech Stack
go
Domain
backenddatabase
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
Go
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30