-
Notifications
You must be signed in to change notification settings - Fork 71
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
capi: execute single transaction #2665
Merged
+922
−61
Merged
Changes from all commits
Commits
Show all changes
48 commits
Select commit
Hold shift + click to select a range
4b1494b
Provisional execution code
JacekGlen 79bcfee
Merge branch 'master' into capi-execute-single-txn
JacekGlen e000cfc
Update remote state
JacekGlen aa9bd06
Merge branch 'master' into capi-execute-single-txn
JacekGlen 3ff8c24
Test run
JacekGlen 3e21459
Make code compile
bzawisto 6102617
Update mdbx
JacekGlen e5d602c
Update single_tx
JacekGlen 06331a0
Merge branch 'master' into capi-execute-single-txn
JacekGlen b97e458
Merge branch 'master' into capi-execute-single-txn
JacekGlen 5d30a9b
disable domain accessor index
battlmonstr c6f356e
remove T prefix from non-template types
battlmonstr cbe767f
rename InvertedIndexRangeByKeyQuery to entity/entity_name for unifica…
battlmonstr 7ae5e3d
Merge branch 'master' into capi-execute-single-txn
JacekGlen 5b1ebc6
move murmur_hash3 to snapshots/common
battlmonstr c368efc
salt file
battlmonstr 225351d
DomainGetLatestQuery
battlmonstr 50c05da
Update to latest code changes
JacekGlen 04d8841
Merge remote-tracking branch 'origin/pr/domain_get_latest2' into capi…
JacekGlen befd4b1
Add DomainState and tests
JacekGlen 34a0efe
make fmt
web-flow e7bc232
Merge branch 'master' into capi-execute-single-txn
JacekGlen ec6a7e5
Update copyright
JacekGlen 4777cb0
Remove temp log comments
JacekGlen 2e159cc
make fmt
web-flow 6ef32fd
Fix test execution
JacekGlen a9f2bde
Merge branch 'capi-execute-single-txn' of https://github.com/erigonte…
JacekGlen 222e78d
Add unit tests and optimize domain state queries
JacekGlen 1736796
Fix silkworm capi test
JacekGlen 8969083
Consistent naming for txn index
JacekGlen c7160b8
Remove redundant debug messages
JacekGlen a3f3f18
Merge branch 'master' into capi-execute-single-txn
JacekGlen 79e12b5
Merge branch 'master' into capi-execute-single-txn
JacekGlen 4f6c8d2
Simplify domain queries
JacekGlen 23676f1
Merge branch 'master' into capi-execute-single-txn
JacekGlen 2ce87c2
Fix CI build errors
JacekGlen 6f77609
Cleanup code
JacekGlen ce38609
make fmt
web-flow eaadfd4
Remove redundant var
JacekGlen 857d3a1
Merge branch 'capi-execute-single-txn' of https://github.com/erigonte…
JacekGlen 0f40861
Merge branch 'master' into capi-execute-single-txn
JacekGlen 5794859
Changes from code review
JacekGlen 4e79e96
make fmt
web-flow d234325
Merge branch 'master' into capi-execute-single-txn
JacekGlen f032bf6
Merge branch 'capi-execute-single-txn' of https://github.com/erigonte…
JacekGlen 40856f6
Fix build
JacekGlen 6bd9213
Changes from code review
JacekGlen 7314d13
Changes from code review
JacekGlen File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We also have this field inside instance.node_settings.chain_config.
I have on my TODO list to split it to be separate from NodeSettings, because we believe that NodeSettings should just only reflect static CLI options.
So even if duplicate - it is a change in a good direction :)