4paradigm / 4paradigm/OpenMLDB
merge the codec in the two places: `hybridse/include/codec/fe_row_codec.h` and `src/codec/codec.h`
Open
code-standards
- Dominant language
- C++
- Stars
- 1.7k
- Forks
- 331
- Avg merge
- 12d 12h
- Merged PRs (30d)
- 1
Description
Duplicate implementations for codecs
Contributor guide
Research direction
The issue points to two files: hybridse/include/codec/fe_row_codec.h and src/codec/codec.h. Start by examining both files to understand the duplicate codec implementations. Look for common functions or classes that can be merged. Check if there are any tests for these codecs to ensure the merge doesn't break functionality. The goal is to eliminate duplication while maintaining the same behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- backend
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100