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.

Sign up for GitHub

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

Jump to bottom

feat: topic secondary index #20

Closed

cablehead wants to merge 1 commit into main from resolved-pr-19

Owner

cablehead commented Oct 28, 2024 •

edited

Loading

Conflict resolved version of PR created by @marvin-j97 #19

Added secondary index partition for topics
Write to index in Store::append as well, using write batch
Index key uses topic + 0xFF + frame ID as encoding
Removed mut condition from Store::append
Use topic index in Store::head
Added basic head test
Change Store::head to only need topic as &str (no heap allocation)


          apply pr-19

821bba4

Collaborator

marvin-j97 commented Oct 28, 2024

I think you squashed the commits

Owner Author

cablehead commented Oct 28, 2024

Yeah. Or rather, I just ran through your full diff and made the changes again to main.. it was easier that way 😆

cablehead closed this

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

Labels

None yet