Skip to content

store/tikv: prepare for moving coprocessor out#22732

Merged
ti-srebot merged 8 commits intopingcap:masterfrom
disksing:copr4
Feb 18, 2021
Merged

store/tikv: prepare for moving coprocessor out#22732
ti-srebot merged 8 commits intopingcap:masterfrom
disksing:copr4

Conversation

@disksing
Copy link
Contributor

@disksing disksing commented Feb 4, 2021

Signed-off-by: disksing i@disksing.com

What problem does this PR solve?

We are planning to move coprocessor related code out of store/tikv. This PR cleans up dependencies ahead.
Part of #22513

What is changed and how it works?

What's Changed:

  • make some types/functions/variables public.
  • move utilities to /util

Check List

Tests

  • Unit test

Release note

  • No release note

Signed-off-by: disksing <i@disksing.com>
@disksing disksing added the sig/transaction SIG:Transaction label Feb 4, 2021
Copy link
Contributor

@AndreMouche AndreMouche left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rest LGTM

Signed-off-by: disksing <i@disksing.com>
Copy link
Contributor

@AndreMouche AndreMouche left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ti-srebot ti-srebot added the status/LGT1 Indicates that a PR has LGTM 1. label Feb 18, 2021
Copy link
Contributor

@youjiali1995 youjiali1995 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ti-srebot ti-srebot added status/LGT2 Indicates that a PR has LGTM 2. and removed status/LGT1 Indicates that a PR has LGTM 1. labels Feb 18, 2021
@disksing
Copy link
Contributor Author

/merge

@ti-srebot ti-srebot added the status/can-merge Indicates a PR has been approved by a committer. label Feb 18, 2021
@ti-srebot
Copy link
Contributor

/run-all-tests

@ti-srebot ti-srebot merged commit a06c22a into pingcap:master Feb 18, 2021
@disksing disksing deleted the copr4 branch February 18, 2021 21:19
RegionCache: s.store.regionCache,
minCommitTSPushed: &s.minCommitTSPushed,
Client: s.store.client,
resolveLite: true,
Copy link
Contributor

@lysu lysu Jul 21, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is it as desired to change resolveLite be false after this change ~?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No. It should be a mistake :(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sig/transaction SIG:Transaction status/can-merge Indicates a PR has been approved by a committer. status/LGT2 Indicates that a PR has LGTM 2.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants