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

Update chainbase to tip (reduce memory usage) #1575

Merged
merged 5 commits into from
Aug 30, 2023
Merged

Update chainbase to tip (reduce memory usage) #1575

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
094308a
Update chainbase to tip (reduce memory usage)
greg7mdp Aug 29, 2023
cf39c5d
Update test to account for chainbase increased capacity.
greg7mdp Aug 29, 2023
cde44e1
Check that configured state db size doesn't exceed chainbase's maximu…
greg7mdp Aug 29, 2023
97d45f0
Fix possible mistake in error message in test.
greg7mdp Aug 29, 2023
c7c9169
Expressly refer to `chain-state-db-size-mb` for user-friendliness.
greg7mdp Aug 29, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion libraries/chainbase
Open in desktop
Submodule chainbase updated 2 files
+41 −17 include/chainbase/undo_index.hpp
+690 −434 test/undo_index.cpp
2 changes: 1 addition & 1 deletion tests/nodeos_under_min_avail_ram.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -144,7 +144,7 @@

# min and max are subjective, just assigned to make sure that many small changes in nodeos don't
# result in the test not correctly validating behavior
if count < 5 or count > 20:
if count < 12 or count > 24:
strMsg="little" if count < 20 else "much"
Utils.cmdError("Was able to send %d store actions which was too %s" % (count, strMsg))
errorExit("Incorrect number of store actions sent")
Expand Down