yugabyte/yugabyte-db

[yb-admin] ysql_catalog_version always returns 0

Open

#11.096 geöffnet am 14. Jan. 2022

Auf GitHub ansehen
 (4 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C (1.003 Forks)batch import
area/documentationgood first issuehelp wantedkind/bugstatus/awaiting-triagetemp/docs-migration

Repository-Metriken

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

Beschreibung

On recent versions, yb-admin ysql_catalog_version keeps returning zero even after catalog changes. This is because it uses CatalogManager::GetYsqlCatalogConfig which reads from the legacy protobuf catalog version, not the pg_yb_catalog_version system table. Fix yb-admin to get the version from the correct source.

Contributor Guide

Research-Richtung
Finden Sie die Implementierung von `GetYsqlCatalogConfig` in der `CatalogManager` Klasse. Verstehen Sie, wie sie derzeit aus der Legacy Protobuf Katalogversion liest. Finden Sie dann heraus, wie die Version aus der Systemtabelle `pg yb catalog version` gelesen werden kann. Die Korrektur sollte `yb admin` so ändern, dass es die richtige Methode aufruft oder die Tabelle direkt abfragt. Überprüfen Sie außerdem vorhandene Tests für `yb admin`, um die Korrektheit sicherzustellen.
Tech Stack
cppsqlpostgresql
Domain
backenddatabase
Issue Type
Fehler
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-3 Stunden
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.
Klar
Voraussetzungen
C++PostgreSQL internalsSQL
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
50