pingcap / pingcap/tidb

Use `IndexReaderExecutor` in `IndexLookUp` and `IndexMerge`.

Open
#43,039 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
In `IndexLookUp` and `IndexMerge`, we didn't create an IndexReaderExecutor which caused a lot of duplicate codes and missed some monitor code.
We could create an `IndexReaderExecutor` just like `tableReaderExecutor` in `IndexMerge` to refine code. And use `Next` function instead of use `result.Next()` directly.

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.