Skip to content

Remove now-unused database code#3486

Open
badboy wants to merge 1 commit into
correctly-validate-usage-of-ke/opmozvvytyzqfrom
sqlite/remove-unused-code
Open

Remove now-unused database code#3486
badboy wants to merge 1 commit into
correctly-validate-usage-of-ke/opmozvvytyzqfrom
sqlite/remove-unused-code

Conversation

@badboy

@badboy badboy commented Jun 8, 2026

Copy link
Copy Markdown
Member

Basically anything that assumes the database layout of rkv, now that it has been reimplemented with sqlite.


This rips out a lot. Not putting this here now would cause me a lot of rebasing and merge conflicts with later commits that I still need to land.

@badboy badboy added the sqlite Any changes to the new SQLite storage backend label Jun 8, 2026
@badboy badboy force-pushed the correctly-validate-usage-of-ke/opmozvvytyzq branch 3 times, most recently from fda2b98 to aca536b Compare June 9, 2026 11:56
@badboy badboy force-pushed the sqlite/remove-unused-code branch from a91b007 to 97a8213 Compare June 9, 2026 12:02
@badboy badboy marked this pull request as ready for review June 9, 2026 12:52
@badboy badboy requested a review from a team as a code owner June 9, 2026 12:52
@badboy badboy requested review from travis79 and removed request for a team June 9, 2026 12:52
Basically anything that assumes the database layout of rkv, now that it
has been reimplemented with sqlite.
@badboy badboy force-pushed the correctly-validate-usage-of-ke/opmozvvytyzq branch from aca536b to 3b99ae5 Compare June 10, 2026 10:12
@badboy badboy force-pushed the sqlite/remove-unused-code branch from 97a8213 to 6d99444 Compare June 10, 2026 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sqlite Any changes to the new SQLite storage backend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant