tikv/tikv

Switch to use codecs in components/codec

Open

#4040 opened on Jan 9, 2019

View on GitHub
 (4 comments) (0 reactions) (0 assignees)Rust (15,922 stars) (2,189 forks)batch import
component/performancedifficulty/easyhelp wantedstatus/mentor

Description

Feature Request

Since https://github.com/tikv/tikv/pull/3629 is landed, we can gradually use the new codec in components/codec instead of src/util/codec, which is more efficient and also easier to use.

Contributor guide

Tech stack
rust
Domain
backendperformance
Issue type
refactor
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-2 days
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
RustCodebase familiarity
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
30
Research direction
Identify all files currently using `src/util/codec` and replace with equivalent functionality from `components/codec`. The new codec is more efficient and easier to use. Start by examining the PR #3629 for patterns and check the components/codec module for the API. Ensure tests pass after changes.