Skip to content
This repository was archived by the owner on Apr 24, 2022. It is now read-only.

New Api server based on Boost #1208

Merged
merged 1 commit into from
Jun 3, 2018
Merged

New Api server based on Boost #1208

merged 1 commit into from
Jun 3, 2018

Conversation

AndreaLanfranchi
Copy link
Collaborator

@AndreaLanfranchi AndreaLanfranchi commented Jun 3, 2018

  • Got rid of jsonrpccpp server which hammers socket (Boost::asio async reads hammering socket and network driver #1059)
  • Some cnotes pushed in log to detect API operations
  • Socket(s) are persistent till disconnection (a client can issue multiple requests)
  • Added miner_shuffle method to re-rand nonce scrambler at runtime (it may give you the chance to abandon a low performance batch)

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
- Got rid of jsonrpccpp server which hammers socket (#1059)
- Some cnotes pushed in log to detect API operations
- Socket(s) are persistent till disconnection (a client can issue
multiple requests)
- Added miner_shuffle method to re-rand nonce scrambler at runtime (it
may give you the chance to abandon a low performance batch)
@AndreaLanfranchi AndreaLanfranchi requested review from jean-m-cyr, smurfy and chfast and removed request for jean-m-cyr and smurfy June 3, 2018 15:09
Copy link
Contributor

@jean-m-cyr jean-m-cyr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

BTW. The is no such thing as a 'low performance batch'! In theory, as well as in practice, every nonce has an equal change of being a valid solution.

@AndreaLanfranchi
Copy link
Collaborator Author

Yes but it's like rolling the dices. Sometimes you want to change hand or stand up if you're on a bad strike. A bit of mental satisfaction

@AndreaLanfranchi AndreaLanfranchi merged commit c6dc335 into master Jun 3, 2018
@AndreaLanfranchi AndreaLanfranchi deleted the api-server branch June 4, 2018 18:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants