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

New index txhash -> txnum or its variation #12366

Open
JkLondon opened this issue Oct 18, 2024 · 0 comments · May be fixed by #12424
Open

New index txhash -> txnum or its variation #12366

JkLondon opened this issue Oct 18, 2024 · 0 comments · May be fixed by #12424
Assignees
@JkLondon
Labels
erigon3 imp2 Medium importance

Comments

@JkLondon
Copy link
Member

JkLondon commented Oct 18, 2024 •
edited
Loading

Rationale

In case of transaction granularity usage we need index that would provide us txnum by only hash (now we only have txhash->blocknum from e2)

Implementation

There are some variants:

  • make new index hash -> txnum
  • change existing index to something like hash->txnum,blocknum

It would provide us a way to implement #12343 etc
@JkLondon JkLondon self-assigned this Oct 18, 2024
@AskAlexSharov AskAlexSharov added the imp2 Medium importance label Oct 21, 2024
@JkLondon JkLondon linked a pull request Oct 23, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JkLondon JkLondon

Labels
erigon3 imp2 Medium importance
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[wip] Txnum index DON'T MERGE erigontech/erigon
2 participants
@AskAlexSharov @JkLondon