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.

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

Use smaller keys for block and transaction database indexes #3152

Closed
Tracked by #3134
mpguerra opened this issue Dec 3, 2021 · 0 comments · Fixed by #3874
Closed
Tracked by #3134

Use smaller keys for block and transaction database indexes #3152

mpguerra opened this issue Dec 3, 2021 · 0 comments · Fixed by #3874
Assignees
@teor2345
Labels
A-devops Area: Pipelines, CI/CD and Dockerfiles A-rust Area: Updates to Rust code C-enhancement Category: This is an improvement lightwalletd any work associated with lightwalletd

Comments

@mpguerra
Copy link
Contributor

mpguerra commented Dec 3, 2021 •
edited by teor2345
Loading

Designs

https://github.com/ZcashFoundation/zebra/blob/main/book/src/dev/rfcs/0005-state-updates.md#rocksdb-data-structures

Tasks

  1. Make the changes
  2. Increment the database version
  3. If you created new Rust modules, make sure the regenerate state paths are up to date in https://github.com/ZcashFoundation/zebra/blob/main/.github/workflows/test.yml
@mpguerra mpguerra mentioned this issue Dec 3, 2021
Closed
33 tasks
@teor2345 teor2345 mentioned this issue Dec 6, 2021
Merged
2 tasks
@teor2345 teor2345 added A-devops Area: Pipelines, CI/CD and Dockerfiles A-rust Area: Updates to Rust code C-enhancement Category: This is an improvement labels Feb 8, 2022
@teor2345 teor2345 mentioned this issue Feb 8, 2022
Merged
@ftm1000 ftm1000 added the S-needs-triage Status: A bug report needs triage label Feb 10, 2022
@ftm1000 ftm1000 removed the S-needs-triage Status: A bug report needs triage label Feb 21, 2022
This was referenced Feb 24, 2022
Closed
Closed
Closed
Merged
Merged
This was referenced Mar 4, 2022
Merged
Merged
Closed
@teor2345 teor2345 self-assigned this Mar 14, 2022
This was referenced Mar 15, 2022
Merged
Merged
@ftm1000 ftm1000 added the lightwalletd any work associated with lightwalletd label Mar 16, 2022
@mergify mergify bot closed this as completed in #3874 Mar 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@teor2345 teor2345

Labels
A-devops Area: Pipelines, CI/CD and Dockerfiles A-rust Area: Updates to Rust code C-enhancement Category: This is an improvement lightwalletd any work associated with lightwalletd
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

7. change(db): Use smaller keys for height and transaction indexes ZcashFoundation/zebra
3 participants
@mpguerra @teor2345 @ftm1000