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

kvstorage: add package #93901

Merged
merged 1 commit into from
Dec 20, 2022
Merged

kvstorage: add package #93901

merged 1 commit into from
Dec 20, 2022

Conversation

tbg
Copy link
Member

@tbg tbg commented Dec 19, 2022

We need a place for logic and testing of the engine interactions for the
separate raft log to live. This is this place.

Epic: CRDB-220

@tbg tbg self-assigned this Dec 19, 2022
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@tbg tbg force-pushed the kvstorage branch 2 times, most recently from 686c891 to 18d0bae Compare December 19, 2022 15:02
@tbg tbg requested a review from a team as a code owner December 19, 2022 15:02
@tbg tbg requested a review from pav-kv December 19, 2022 20:56
@tbg
Copy link
Member Author

tbg commented Dec 19, 2022

Note: a follow-up PR is moving code into this package, #93904.

@tbg tbg removed their assignment Dec 19, 2022
We need a place for logic and testing of the engine interactions for the
separate raft log to live. This is this place.

Epic: CRDB-220
@tbg
Copy link
Member Author

tbg commented Dec 20, 2022

TFTR!

bors r=pavelkalinnikov

@craig
Copy link
Contributor

craig bot commented Dec 20, 2022

Build succeeded:

@craig craig bot merged commit 6eddf3e into master Dec 20, 2022
@craig craig bot deleted the kvstorage branch December 20, 2022 12:06
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.

3 participants