pingcap/tiflow

Move the common logic together with BR.

Aperta

#956 aperta il 17 set 2020

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)Go (310 fork)auto 404
help wanted

Metriche repository

Star
 (461 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Feature Request

Is your feature request related to a problem? Please describe:

According to https://github.com/pingcap/br/pull/482#discussion_r488934395, BR and CDC both have the same variables and interfaces in different repo which increase the maintenance cos.

Describe the feature you'd like:

we can move the decoder logic together by

  1. import BR cause CDC already import BR as lib.
  2. move decoder into a third-part repo.

Describe alternatives you've considered:

Teachability, Documentation, Adoption, Migration Strategy:

Guida contributor

Direzione di ricerca
Indaga la logica del decoder condivisa tra BR e CDC nel repository tiflow e proponi un piano per spostarla in una posizione comune.
Tech stack
go
Dominio
backenddatabase
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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
GoGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
55