-
Notifications
You must be signed in to change notification settings - Fork 114
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
integrity check of local db #593
Comments
This was referenced Jun 28, 2019
seems done, should close? @qizhou |
May need the last steps of checking xshard txs. |
what should we check for xshard txs? in my understanding, |
We should check the xshard deposits on target shard matches the one of the
local shard.
- Qi
…On Tue, Aug 6, 2019 at 12:04 PM ninjaahhh ***@***.***> wrote:
what should we check for xshard txs? in my understanding, CheckMinorBlock
already checks 1) processing xshard deposits; 2) processing xshard tx on
the sender side
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#593?email_source=notifications&email_token=AATMNZT3UNFOJFDHJ3PPXZLQDHDMNA5CNFSM4H4HUTG2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3WEULA#issuecomment-518801964>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AATMNZVYNKHTF437HRQF2STQDHDMNANCNFSM4H4HUTGQ>
.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Make sure if we change the code, the code can recognize local db, i.e., we don't break the existing consensus.
A couple of checks:
etc.
The text was updated successfully, but these errors were encountered: