yugabyte/yugabyte-db

[yb-admin] ysql_catalog_version always returns 0

Open

#11 096 ouverte le 14 janv. 2022

Voir sur GitHub
 (4 commentaires) (0 réactions) (0 assignés)C (1 003 forks)batch import
area/documentationgood first issuehelp wantedkind/bugstatus/awaiting-triagetemp/docs-migration

Métriques du dépôt

Stars
 (8 229 stars)
Métriques de merge PR
 (Merge moyen 17j 21h) (92 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Localisez l'implémentation de `GetYsqlCatalogConfig` dans la classe `CatalogManager`. Comprenez comment elle lit actuellement à partir de l'ancienne version du catalogue protobuf. Trouvez ensuite comment lire la version à partir de la table système `pg yb catalog version`. La correction doit modifier `yb admin` pour appeler la méthode correcte ou interroger directement la table. Vérifiez également les tests existants pour `yb admin` pour garantir l'exactitude.
Stack technique
cppsqlpostgresql
Domaine
backenddatabase
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++PostgreSQL internalsSQL
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50