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 an RemoveFromBuffer method for test #387

Merged
merged 4 commits into from
Nov 30, 2021

Conversation

ekexium
Copy link
Contributor

@ekexium ekexium commented Nov 29, 2021

Removing a record from the membuffer can be useful for test, e.g. in pingcap/tidb#26833.

Signed-off-by: ekexium <ekexium@gmail.com>
Signed-off-by: ekexium <ekexium@gmail.com>
Signed-off-by: ekexium <ekexium@gmail.com>
@ekexium
Copy link
Contributor Author

ekexium commented Nov 30, 2021

I just found it's already in #311 🥲

@MyonKeminta
Copy link
Contributor

I just found it's already in #311 🥲

I think let's adopt this one 😂

@MyonKeminta
Copy link
Contributor

PTAL @cfzjywxk
Can someone add you to the Github organization? We can't add you to the reviewers of PRs 😂

Copy link
Contributor

@cfzjywxk cfzjywxk left a comment

Choose a reason for hiding this comment

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

LGTM

@MyonKeminta MyonKeminta changed the title Add an UnsafeRemoveRecord method for test Add an RemoveFromBuffer method for test Nov 30, 2021
@MyonKeminta
Copy link
Contributor

/merge

@sticnarf sticnarf merged commit 350b6ce into tikv:master Nov 30, 2021
MyonKeminta pushed a commit to MyonKeminta/client-go that referenced this pull request Dec 13, 2021
* export DeleteKey

Signed-off-by: ekexium <ekexium@gmail.com>

* rename DeleteKey to UnsafeRemoveRecord

Signed-off-by: ekexium <ekexium@gmail.com>

* rename the old DeleteKey

Signed-off-by: ekexium <ekexium@gmail.com>
Signed-off-by: MyonKeminta <MyonKeminta@users.noreply.github.com>
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.

4 participants