yugabyte/yugabyte-db

docdb: move TableNameResolver to pgwrapper

Open

#6868 aperta il 13 gen 2021

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)C (1003 fork)batch import
area/docdbgood first issuekind/enhancementpriority/medium

Metriche repository

Star
 (8229 star)
Metriche merge PR
 (Merge medio 17g 21h) (92 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Individua la classe TableNameResolver nella directory del codice yb admin. Quindi spostala nella directory pgwrapper (probabilmente sotto src/yb/client o src/yb/pgwrapper). Aggiorna i percorsi di inclusione e il namespace se necessario. Sostituisci la funzione helper `GetTableIdByTableName` in `PgLibPqTest` con il TableNameResolver spostato. Assicurati che tutti i test passino. Rivedi eventuali commenti esistenti nell'issue per ulteriore contesto.
Tech stack
cpp
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30