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

Store timestamp & verified signature status in unchecked table #1492

Merged
merged 91 commits into from Jan 30, 2019

Conversation

Projects
3 participants
@SergiySW
Copy link
Collaborator

commented Dec 21, 2018

Additionally periodic unchecked_cleaning ()

This replaces #1459

@SergiySW SergiySW self-assigned this Dec 21, 2018

SergiySW added some commits Dec 21, 2018

@rkeene rkeene added the enhancement label Dec 21, 2018

@rkeene rkeene added this to the V18.0 milestone Dec 21, 2018

@rkeene rkeene self-requested a review Dec 21, 2018

@zhyatt zhyatt added this to Unscheduled in V18 Dec 27, 2018

@SergiySW

This comment has been minimized.

Copy link
Collaborator Author

commented Dec 28, 2018

--debug_profile_bootstrap shows 47% improvements for beta network with no significant changes on live network due to slow write/read for large tables

@SergiySW SergiySW moved this from Unscheduled to CP 2 (2018-01-16) in V18 Dec 28, 2018

@zhyatt zhyatt requested review from clemahieu and wezrule Jan 29, 2019

Show resolved Hide resolved nano/secure/ledger.cpp Outdated
Show resolved Hide resolved nano/secure/ledger.cpp
Show resolved Hide resolved nano/secure/common.cpp Outdated
Show resolved Hide resolved nano/secure/common.cpp Outdated
Show resolved Hide resolved nano/node/node.cpp

rkeene and others added some commits Jan 29, 2019

Apply suggestion from code review
Co-Authored-By: SergiySW <sergiysw@gmail.com>
Show resolved Hide resolved nano/secure/ledger.cpp Outdated
@rkeene

rkeene approved these changes Jan 29, 2019

Copy link
Contributor

left a comment

Thanks !

SergiySW added some commits Jan 29, 2019

Fix

@SergiySW SergiySW merged commit e6ab82f into nanocurrency:master Jan 30, 2019

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.