yugabyte/yugabyte-db

[DocDB] GetConsensusState could also return current index.

Open

#13.702 geöffnet am 21. Aug. 2022

Auf GitHub ansehen
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C (1.003 Forks)batch import
area/docdbgood first issuekind/enhancementpriority/medium

Repository-Metriken

Stars
 (8.229 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 17T 21h) (92 gemergte PRs in 30 T)

Beschreibung

Jira Link: DB-3242

Description

GetConsensusState is used by scripts such as yugatool to generate tablet-reports.

However the information printed wrt term/index are inaccurate:

The index printed which comes from RaftConfigPB corresponds to the index at which the last config-change took place.

It would be more relevant to have the current-index information be added to the proto. Also, for the leader, it would be useful to add information about whether or not the initial no-op has been replicated.

cc : @tvesely @eugeneckim @tylarb

Contributor Guide

Research-Richtung
Untersuche die aktuelle GetConsensusState Protodefinition und verfolge, wie term/index aus RaftConfigPB bezogen werden. Ändere das Proto, um das aktuelle Indexfeld und das No Op Flag des Leaders einzufügen. Stelle die Abwärtskompatibilität sicher.
Tech Stack
c
Domain
backend
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.
Halber Tag
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
CProtobufRaft
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40