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

evm integration for transaction execution #35

Open
arcolife opened this issue Jul 10, 2018 · 1 comment
Open

evm integration for transaction execution #35

arcolife opened this issue Jul 10, 2018 · 1 comment
Assignees
Labels
A-level non trivial issues with high difficulty or repo maintainers bug Something isn't working
Milestone

Comments

@arcolife
Copy link
Collaborator

to facilitate payload execution, we need compiler integration. Related to #31

cc/ @samikshan

@arcolife arcolife added the bug Something isn't working label Jul 10, 2018
@arcolife arcolife added this to the Alpha 1 milestone Jul 10, 2018
@arcolife arcolife self-assigned this Jul 10, 2018
@arcolife arcolife modified the milestones: Alpha 1, Alpha 2 Jul 10, 2018
@arcolife arcolife added the A-level non trivial issues with high difficulty or repo maintainers label Jul 24, 2018
@arcolife
Copy link
Collaborator Author

func (nd *Node) execute(am ApplyMsg) {	
	// TODO: add msg to applyCh, should be executed in a separate go routine	
	// TODO: add we probably want to keep a log for this	
	nd.ApplyCh <- am	
}

..is the bane of transaction execution right now. Removed in #61 but needs to be brought back in this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-level non trivial issues with high difficulty or repo maintainers bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants