Skip to content

refactor: catchpointdb #5444

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
Draft
wants to merge 69 commits into
base: master
Choose a base branch
from
Draft

refactor: catchpointdb #5444

wants to merge 69 commits into from

Conversation

@icorderi
Copy link
Contributor

@icorderi icorderi commented Jun 2, 2023

Summary

  • Refactors catchpoint data into a separate db.
  • Creates a catchpoint application interface.

Test Plan

Existing tests.

icorderi added 30 commits June 2, 2023 10:51
@icorderi
cleanup: remove redundant method LookupAccountDataByAddress
98106ac
@icorderi
refactor: upgrade MakeAccountsOptimizedReader from a testing only m…
717995b 
…ethod.
@icorderi
misc: import pebble module
d53d749
@icorderi
feat: add storage engine to the config
252c58b
@icorderi
feat: adjustments to the store interface
6988a2b
@icorderi
feat: update sql impl based on latest interface changes
69f6000
@icorderi
feat: a generic KV impl of store
0d56ded
@icorderi
feat: pebble impl using generickv
55a8bf5
@icorderi
feat: mockdb and multi engine testing suite
95fe47f
@icorderi
test: add new db tests to compare impls
f674291
@icorderi
misc: adjust make file for msp
ef7d0cc
@icorderi
fix: different ErrNotFound by engine
72f8ab8
@icorderi
feat: split tracker init to aavoid read-after-write txn
dc688d3
@icorderi
feat: use StorageEngine config during ledger init
a9a917c
@icorderi
linter
93f180e
@icorderi
fix: config tests
e4cf417
@icorderi
misc: remove demo comments
9931831
@icorderi
misc: add missing license
836f967
@icorderi
fix: unsaved files
463ea43
@icorderi
fix: migrations always run on startup
f7d2cb3
@icorderi
misc: send proto from the test to the engines
b0815bb
@icorderi
fix: LookupOnline not matching behaviour
0859aa5
@icorderi
fix: LookupOnlineHistory
165a2c5
@icorderi
test: online accounts data by address
028d2a0
@icorderi
fix: LookupOnlineTotalsHistory for kv
9dc2b68
@icorderi
fix: OnlineAccountsDelete for KV
c27f9bd
@icorderi
fix: LookupAccountRowID for KV
c119d49
@icorderi
test: ResourceLookupByRowID
8f7c1aa
@icorderi
fix: failing test due to sqlNoRows removal
89a5dfd
@icorderi
misc: default pebble options
7790321
icorderi added 25 commits June 2, 2023 10:54
@icorderi
refactor: drivers file rename and cleanup
a8ea766
@icorderi
refactor: use Open to create dbs
2f2584a
@icorderi
refactor: remove from the interface
a6d2f1b
@icorderi
refactor: remove SetLogger from Store interface
7d6c988
@icorderi
refactor: reduce the number of RunMigrations called inside transact…
8a512ed 
…ions
@icorderi
feat: impl IsSharedCacheConnection for kv
4e8a93d
@icorderi
s: pebble import
44ce944
@icorderi
s pebble
c9f39ad
@icorderi
linter
747b439
@icorderi
fix: migration
aaa9a80
@icorderi
fix test
af6e952
@icorderi
fix: close cannot rollback if commited on sql
7b5a52c
@icorderi
comment: avoid copy on for loop
678497b
@icorderi
misc: remove unused code
5057025
@icorderi
linter
d260e2e
@icorderi
fix: list creatables cidx key parsing
8a48385
@icorderi
fix: aidx in LookupAllResources
a61ea76
@icorderi
fix: hex encoding of address in keys
a85722b
@icorderi
misc: use geth pebble config
373f059
@icorderi
feat: add supprt for new online stake method
3d57a90
@icorderi
go.mod
8ecac02
@icorderi
fix: block-generator tidy
32d556e
@icorderi
remove unnecesary imports
52c7ca7
@icorderi
fix: shadows
e53c6ca
@icorderi
refactor: extract catchpoint data into a separate db
c01c356
@icorderi icorderi self-assigned this Jun 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@icorderi icorderi

Labels

Skip-Release-Notes Team Carbon-11

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@icorderi @jsgranados