tispark icon indicating copy to clipboard operation
tispark copied to clipboard

Optimize index read logic

Open birdstorm opened this issue 5 years ago • 1 comments

Current index read contains some logic that is confusing, e.g., covering index and double read use the same RegionExecTask, they could be better distinguished since they share a same children HandleExec.

Also, the logic in index read could be optimized by partitioning the handles according to region ids beforehand, so that it should reduce the cost of retrying when any RegionError occurs.

Some tasks to be done:

  • [ ] refine index read logic to separate covering index from index read.
  • [ ] optimize index read to partition handles by region ids.
  • [ ] eliminate confusing logic in double read logic and make explain result more meaningful.

@zhexuany @marsishandsome PTAL

birdstorm avatar Nov 11 '19 08:11 birdstorm

This issue is referenced to https://github.com/pingcap/tispark/pull/1195

zhexuany avatar Dec 12 '19 15:12 zhexuany