yugabyte/yugabyte-db

docdb: move TableNameResolver to pgwrapper

Open

#6.868 geöffnet am 13. Jan. 2021

Auf GitHub ansehen
 (1 Kommentar) (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-1650 TableNameResolver by @d-uspenskiy looks useful for tests, but it exists in yb-admin code. How about we move it to client so that tests can use it. For example, PgLibPqTest has its own helper GetTableIdByTableName, and TableNameResolver can be used instead.

Contributor Guide

Research-Richtung
Suchen Sie die Klasse TableNameResolver im yb admin Codeverzeichnis. Verschieben Sie sie dann in das pgwrapper Verzeichnis (wahrscheinlich unter src/yb/client oder src/yb/pgwrapper). Aktualisieren Sie bei Bedarf die Include Pfade und den Namespace. Ersetzen Sie die Hilfsfunktion `GetTableIdByTableName` in `PgLibPqTest` durch die verschobene TableNameResolver Klasse. Stellen Sie sicher, dass alle Tests bestehen. Überprüfen Sie vorhandene Kommentare im Issue für zusätzlichen Kontext.
Tech Stack
cpp
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
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.
Klar
Voraussetzungen
C++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30