pingcap/tiflow

Move the common logic together with BR.

Offen

#956 geöffnet am 17.09.2020

Auf GitHub ansehenContributor Guide
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Go (310 Forks)auto 404
help wanted

Repository-Metriken

Stars
 (461 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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:

Contributor Guide

Research-Richtung
Untersuchen Sie die gemeinsame Decoder Logik zwischen BR und CDC im tiflow Repository und schlagen Sie einen Plan vor, um sie an einen gemeinsamen Ort zu verschieben.
Tech Stack
go
Domain
backenddatabase
Issue Type
Funktion
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.
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
GoGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
55