4paradigm / 4paradigm/OpenMLDB
merge the codec in the two places: `hybridse/include/codec/fe_row_codec.h` and `src/codec/codec.h`
Đang mở
code-standards
- Ngôn ngữ chính
- C++
- Star
- 1.7k
- Fork
- 331
- Merge trung bình
- 12 ngày 12 giờ
- Pull request đã merge (30 ngày)
- 1
Mô tả
Duplicate implementations for codecs
Hướng dẫn đóng góp
Hướng nghiên cứu
Issue này chỉ ra hai tệp: hybridse/include/codec/fe_row_codec.h và src/codec/codec.h. Trước tiên, hãy xem xét cả hai tệp để hiểu các triển khai codec bị trùng lặp. Tìm các hàm hoặc lớp dùng chung có thể được hợp nhất. Kiểm tra xem có bài kiểm thử nào cho các codec này hay không để đảm bảo việc hợp nhất không làm hỏng chức năng. Mục tiêu là loại bỏ sự trùng lặp trong khi vẫn duy trì cùng một hành vi.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- cpp
- Lĩnh vực
- backend
- Loại issue
- Tái cấu trúc
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 45/100