Closed JaySon-Huang closed 2 weeks ago
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: CalvinNeo, Lloyd-Pottiger
The full list of commands accepted by this bot can be found here.
The pull request process is described here
Timeline:
2024-07-05 09:33:36.141627779 +0000 UTC m=+4513.376861891
: :ballot_box_with_check: agreed by Lloyd-Pottiger.2024-07-05 11:27:00.459107998 +0000 UTC m=+11317.694342108
: :ballot_box_with_check: agreed by CalvinNeo.
What problem does this PR solve?
Issue Number: ref https://github.com/pingcap/tiflash/issues/8081
Problem Summary:
I'd like to introduce a new layer
SnapshotSSTReader
and movegetApproxBytes
/findSplitKeys
into that layer. This could make the logic in parallel prehandle snapshots more clear.This PR is a preceding PR that refactors the code to simplify the following logical change.
New
SnapshotSSTReader
interface in later PR:What is changed and how it works?
extra_id
bysplit_id
executeParallelTransform
andexecuteTransform
to bestd::tuple<ReadFromStreamResult, PrehandleResult>
executeParallelTransform
to simplify the code indent layerCheck List
Tests
Side effects
Documentation
Release note