-
Notifications
You must be signed in to change notification settings - Fork 42
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
MacroSync #16
MacroSync #16
Conversation
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.
Fix get_epoch_transactions
.
Some are duplicate. I had close the browser and thought they were lost. |
Doesn't need fixing, but I will take care of the style issues. |
a47a6dc
to
b28797d
Compare
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.
👍
b28797d
to
ea24cfa
Compare
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.
Stil 👍 also after rebasing.
Implements the MacroSync protocol into the client.
I haven't integrated it with the config though, since @jgraef changed the config builder in one of his branches completely and I didn't want to do the work twice.
For now, it always uses the full sync client and the behaviour shouldn't be any different from the current one.
The main two TODOs, for which I will open issues after having merged this PR are:
Otherwise a client can send us any subset of the transactions and we will only find out that our vector doesn't give the correct hash when he is done sending us transactions.