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.

Sign up for GitHub

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

Jump to bottom

execution result consensus #1712

Closed
wants to merge 3 commits into from
Closed

execution result consensus #1712

wants to merge 3 commits into from

Conversation

krhubert
Copy link
Contributor

@krhubert krhubert commented Mar 6, 2020

@krhubert krhubert added the enhancement New feature or request label Mar 6, 2020
@krhubert krhubert self-assigned this Mar 6, 2020
@antho1404
Copy link
Member

This PR was not planned and also definitely not in was we are supposed to do...
There is no consensus on the execution, only one executor will be selected (maybe multiple in case the previous doesn't respond but this is just a fallback).
We will have multiple verifications of this single result but that's not the execution part.

Also, even if this was the direction, this PR would be wrong because we have a consensus on only one selected executor in the execution.

@NicolasMahe
Copy link
Member

closing this PR as it's not the direction of current development

@NicolasMahe NicolasMahe deleted the feature/execution-result-consensus branch March 12, 2020 04:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@antho1404 antho1404 Awaiting requested review from antho1404

@NicolasMahe NicolasMahe Awaiting requested review from NicolasMahe

Assignees

@krhubert krhubert

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@krhubert @antho1404 @NicolasMahe