4paradigm / 4paradigm/OpenMLDB

merge the codec in the two places: `hybridse/include/codec/fe_row_codec.h` and `src/codec/codec.h`

Ouverte
#1,534 0 commentaires 0 réactions 1 personne assignée Réclamée par @zhanghaohit Voir sur GitHub
code-standards
Langage dominant
C++
Étoiles
1.7k
Forks
331
Merge moyen
12 j 12 h
PR mergées (30 j)
1

Description

Duplicate implementations for codecs

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

L’issue pointe vers deux fichiers : hybridse/include/codec/fe_row_codec.h et src/codec/codec.h. Commencez par examiner les deux fichiers afin de comprendre les implémentations de codec dupliquées. Recherchez les fonctions ou classes communes qui peuvent être fusionnées. Vérifiez s’il existe des tests pour ces codecs afin de vous assurer que la fusion ne casse pas les fonctionnalités. L’objectif est d’éliminer la duplication tout en conservant le même comportement.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
cpp
Domaine
backend
Type d'issue
Refactorisation
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
45/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.