Refactor package `backend` to decouple interfaces
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 45/100
Piste de recherche
Commencez dans le package backend en retraçant comment BackendConnMgr appelle Authenticator et CmdProcessor, puis examinez les tests unitaires existants. Définissez les interfaces Auth et CmdEngine autour du comportement requis et mettez à jour le manager pour utiliser des implémentations, y compris des implémentations mock dans les tests. Le travail est terminé lorsque les tests unitaires peuvent mocker les deux collaborateurs et que des implémentations alternatives de plugins peuvent les fournir.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Development Task
Currently, BackendConnMgr calls structs Authenticator and CmdProcessor directly, which makes them coupled. It makes UT more complicated, and it will be hard to extend this module. E.g, we may override some behaviors through plugins.
We can define interface Auth for Authenticator and CmdEngine for CmdProcessor. UT can mock implementations and plugins can override the default implementation by defining another implementation.
- Langage dominant
- Go
- Étoiles
- 73
- Forks
- 41
- Merge moyen
- 4 h 13 min
- PR mergées (30 j)
- 7
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de pingcap/tiproxy
-
migration:bb7133-tiproxy
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 1 jour
-
migration:bb7133-tiproxy
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 1 jour
-
[Rust-dataplane] Migration backend handshake must preserve CONNECT_WITH_DB when current-db emptyOuvertemigration:bb7133-tiproxy
Difficulté 2/5 1-3 heures Accessibilité débutants 64/100
Les mainteneurs répondent en général sous 1 jour
-
[Rust-dataplane] Migration must not force CONNECT_ATTRS on backend lacking it with empty attrsOuvertemigration:bb7133-tiproxy
Difficulté 2/5 1-3 heures Accessibilité débutants 77/100
Les mainteneurs répondent en général sous 1 jour
-
migration:bb7133-tiproxy
Difficulté 1/5 Moins d'une heure Accessibilité débutants 82/100
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de pingcap/tiproxy
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
grafana/tempo-operator#1691 ·
Les mainteneurs répondent en général sous 1 jour
-
enhancement
Difficulté 1/5 Moins d'une heure Accessibilité débutants 85/100
Les mainteneurs répondent en général sous 1 jour
-
search_modules reports "failed to parse module results" when the registry returns no modulesOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
hashicorp/terraform-mcp-server#583 ·
Les mainteneurs répondent en général sous 3 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
-
backend bug low-priority
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
lognorman20/monaco#6225 ·