Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

*: add getPDClient to rawKV public api #570

Merged
merged 1 commit into from
Aug 25, 2022
Merged

Conversation

c4pt0r
Copy link
Contributor

@c4pt0r c4pt0r commented Aug 24, 2022

TxnAPI exposes GetPDClient
To ensure consistency in the semantics of the API, the rawkv API should also expose it

@disksing disksing merged commit 46dcf45 into tikv:master Aug 25, 2022
TonsnakeLin pushed a commit to TonsnakeLin/client-go that referenced this pull request Aug 31, 2022
Signed-off-by: dongxu <[email protected]>

Signed-off-by: dongxu <[email protected]>
Signed-off-by: TonsnakeLin <[email protected]>
cfzjywxk added a commit that referenced this pull request Sep 6, 2022
* optimize for lock if exists

Signed-off-by: TonsnakeLin <[email protected]>

* fix bugs for lock if exists

Signed-off-by: TonsnakeLin <[email protected]>

* optimize lock info

Signed-off-by: TonsnakeLin <[email protected]>

* fix bugs

Signed-off-by: TonsnakeLin <[email protected]>

* remove the contrl flag for lock stats

Signed-off-by: TonsnakeLin <[email protected]>

* update kvproto

Signed-off-by: TonsnakeLin <[email protected]>

* change to LockOnlyIfExists

Signed-off-by: TonsnakeLin <[email protected]>

* update kvproto

Signed-off-by: TonsnakeLin <[email protected]>

* change to LockOnlyIfExists

Signed-off-by: TonsnakeLin <[email protected]>

* make test pass

Signed-off-by: TonsnakeLin <[email protected]>

* Update txnkv/transaction/txn.go

Co-authored-by: MyonKeminta <[email protected]>
Signed-off-by: TonsnakeLin <[email protected]>

* *: add getPDClient to rawKV public api (#570)

Signed-off-by: dongxu <[email protected]>

Signed-off-by: dongxu <[email protected]>
Signed-off-by: TonsnakeLin <[email protected]>

* Update README.md (#571)

Signed-off-by: dongxu <[email protected]>
Signed-off-by: TonsnakeLin <[email protected]>

* update dependency of integration test (#572)

Signed-off-by: cfzjywxk <[email protected]>

Signed-off-by: cfzjywxk <[email protected]>
Signed-off-by: TonsnakeLin <[email protected]>

* Support postponed conflict check (#556)

* replace kvproto

Signed-off-by: ekexium <[email protected]>

* support NeedConflictCheck

Signed-off-by: ekexium <[email protected]>

* fix mutation encoding

Signed-off-by: ekexium <[email protected]>

* support temporary flag

Signed-off-by: ekexium <[email protected]>

* update kvproto

Signed-off-by: ekexium <[email protected]>

* fix style

Signed-off-by: ekexium <[email protected]>

* add an option to enable the behavior

Signed-off-by: ekexium <[email protected]>

* replace AfterCheckPoint with existing canModity

Signed-off-by: ekexium <[email protected]>

* UpdateFlag do not unset temporary flag

Signed-off-by: ekexium <[email protected]>

* remove unused function

Signed-off-by: ekexium <[email protected]>

* update tidb dependency

Signed-off-by: ekexium <[email protected]>

update tidb dependency

Signed-off-by: ekexium <[email protected]>

* fix test

Signed-off-by: ekexium <[email protected]>

* do no unset flag on read

Signed-off-by: ekexium <[email protected]>

* update tidb dependency

Signed-off-by: ekexium <[email protected]>

* update comment

Signed-off-by: ekexium <[email protected]>

Signed-off-by: ekexium <[email protected]>

* add testcase

Signed-off-by: TonsnakeLin <[email protected]>

* add test case

Signed-off-by: TonsnakeLin <[email protected]>

* unset pk if lockifexits failed

Signed-off-by: TonsnakeLin <[email protected]>

* fix format

Signed-off-by: TonsnakeLin <[email protected]>

* LockOnlyIfExists only when pk selected

Signed-off-by: TonsnakeLin <[email protected]>

* remove test function to txn_probe

Signed-off-by: TonsnakeLin <[email protected]>

* add more info to error

Signed-off-by: TonsnakeLin <[email protected]>

* add more info to error

Signed-off-by: TonsnakeLin <[email protected]>

* add protection for input key

Signed-off-by: TonsnakeLin <[email protected]>

Signed-off-by: TonsnakeLin <[email protected]>
Signed-off-by: dongxu <[email protected]>
Signed-off-by: cfzjywxk <[email protected]>
Signed-off-by: ekexium <[email protected]>
Co-authored-by: MyonKeminta <[email protected]>
Co-authored-by: dongxu <[email protected]>
Co-authored-by: cfzjywxk <[email protected]>
Co-authored-by: ekexium <[email protected]>
Co-authored-by: Yilin Chen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants