4paradigm / 4paradigm/OpenMLDB

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

オープン
#1,534 コメント 0 件 リアクション 0 件 担当者 1 名 @zhanghaohit が担当を希望しています GitHub で見る
code-standards
主要言語
C++
スター
1.7k
フォーク
331
平均マージ
12日 12時間
マージ済み PR(30日)
1

説明

Duplicate implementations for codecs

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

この issue は 2 つのファイルを指しています: hybridse/include/codec/fe_row_codec.h と src/codec/codec.h。まず両方のファイルを調べ、重複している codec 実装を理解します。統合できる共通の関数またはクラスを探します。統合によって機能が損なわれないことを確認するため、これらの codec 用のテストがあるか確認します。目標は、同じ動作を維持しながら重複をなくすことです。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
cpp
領域
backend
issue の種類
リファクタリング
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
45/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。