pingcap / pingcap/tidb

Refactor coprocessor

Open
#38,448 0 comments 0 reactions 0 assignees View on GitHub
type/enhancement
Dominant language
Go
Stars
40.5k
Forks
6.2k
PR merge metrics
PR metrics pending

Description

## Enhancement

I'm making some changes to the coprocessor recently and found that the code of features is tightly coupled. This issue intends to make the code of the coprocessor clear from the following perspective.

- Separate the iterator build path of TiFlash, TiKV, and TiDB
- It needs to iterate over the regions/buckets when building tasks, making it expandable
- The result of OOM's test is fragile when the code is changed
- Increase the test coverage of the coprocessor, it opens goroutines for concurrency but no tests can guarantee the goroutines works as expected

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.