Skip to content
This repository has been archived by the owner on Oct 4, 2019. It is now read-only.

Pluggable consensus (develop cache) #647

Closed
wants to merge 86 commits into from

Conversation

r8d8
Copy link
Member

@r8d8 r8d8 commented Jul 11, 2018

No description provided.

r8d8 and others added 30 commits June 5, 2018 17:20
solution: add unused Engine() method
solution: use original StartRecord method
solution: import core/types
solution: shopt -s globstar && sed -i 's/vm\.Logs/\[\]\*types\.Log/g' ./**/*.go
solution: sed vm.Log -> *types.Log
solution: sed vm.Account -> vm.AccountRef
solution: modify and add methods to fit
solution: merge tzdybal's upstream/feature/tx-status changes,
resolving some conflicts and creating some conflicts
solution: remove duplicate methods
solution: just move logic to NewReceipt fn
solution: remove unused imports and invalid methods, fix calls to types.Log
solution: export it

* IMO This is really fuckin ugly. TODO, unexport, refactor
solution: fix

- use vm.Config in blockchain#StateProcessor signature
- change returned gas value to uint64 type for ^ method
- reference params.TxGas
- ...
@r8d8 r8d8 changed the title Issue/pluggable consensus Pluggable consensus (develop cache) Jul 11, 2018
@ghost
Copy link

ghost commented Jul 12, 2018

So you guys are implementing POA consensus on classic geth?? Cool 👍

@r8d8
Copy link
Member Author

r8d8 commented Jul 14, 2018

@eosclassicteam, you are right

@r8d8 r8d8 force-pushed the issue/pluggable_consensus branch 2 times, most recently from f09cd95 to 2c3d7bb Compare July 16, 2018 11:24
@ghost
Copy link

ghost commented Jul 17, 2018

@r8d8 Is clique engine compatible with original go-ethereum? Is there any testnet running with it??

@r8d8
Copy link
Member Author

r8d8 commented Jul 17, 2018

@eosclassicteam, yes it is compatible.
testnet - not yet.

@r8d8
Copy link
Member Author

r8d8 commented Jul 17, 2018

Moved to #648

@r8d8 r8d8 closed this Jul 17, 2018
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

3 participants