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

statemachine: Error messages should release (optionally)? #132

Open
schmichael opened this issue Jul 7, 2015 · 0 comments
Open

statemachine: Error messages should release (optionally)? #132

schmichael opened this issue Jul 7, 2015 · 0 comments

Comments

@schmichael
Copy link
Contributor

Problem

There are a number of circumstances where a task cannot be executed on a node that has claimed it:

  • A new task type is being added via a rolling deploy and a node without the necessary code has claimed the a new task type.
  • A task cannot complete due to a node-specific condition: eg a node has insufficient diskspace for a task to use as scratch.

Possible Solution 1: Errors always release

Seems like the worst case scenario to Error messages always causing releases is extra broker load. Probably worth it.

Possible Solution 2: Make Release metadata on a Message

Allow any message to cause a release via a Release bool field on the statemachine.Message struct.

This might actually simplify other code... not sure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant