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

e3: reduce size of domain/hist/ii files. remove history concept from api. #12621

Merged
merged 44 commits into from
Nov 8, 2024

Conversation

AskAlexSharov
Copy link
Collaborator

@AskAlexSharov AskAlexSharov commented Nov 5, 2024

we decided to not expose History concept to users:

  • HistoryRange and HistorySeek methods to accept domain. remove concept of standalone history (and their names).

Reduce size of domain.go/history.go/inverted_index.go:

  • move streams to own files

  • use same scanDirty func

  • remove DomainStats - unused

@AskAlexSharov AskAlexSharov added this to the 3.0.0-alpha6 milestone Nov 8, 2024
@AskAlexSharov AskAlexSharov changed the title [rpc-test] erigon_ namespace e3: renamings Nov 8, 2024
@AskAlexSharov AskAlexSharov enabled auto-merge (squash) November 8, 2024 05:53
@AskAlexSharov AskAlexSharov changed the title e3: renamings e3: renamings and split files Nov 8, 2024
@AskAlexSharov AskAlexSharov changed the title e3: renamings and split files e3: reduce size of domain/hist/ii files. remove history concept from api. Nov 8, 2024
@AskAlexSharov AskAlexSharov merged commit 280bef9 into main Nov 8, 2024
14 of 15 checks passed
@AskAlexSharov AskAlexSharov deleted the erigon_namespace_rpcd branch November 8, 2024 06:51
AskAlexSharov added a commit that referenced this pull request Nov 16, 2024
Rename: 
- `DomainRange` to `RangeAsOf`
- `DomainGetAsOf` to `GetAsOf`
-  `DomainGet` to `GetLatest`
all this methods do accept typed `kv.Domain` as a parameter

merge after #12621
AskAlexSharov added a commit that referenced this pull request Nov 17, 2024
Rename: 
- `DomainRange` to `RangeAsOf`
- `DomainGetAsOf` to `GetAsOf`
-  `DomainGet` to `GetLatest`
all this methods do accept typed `kv.Domain` as a parameter

merge after #12621
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.

2 participants