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

Feature assignment index tracking #6

Merged
merged 1 commit into from
Oct 21, 2015

Conversation

ctrimble
Copy link
Contributor

Not waiting for etcd state updates after a decision causes wild state changes while nodes enter or exit the cluster. This change forces the assignment service to wait for changes to be reflected in etcd before more decisions can be made. It also adds a new document in etcd for each running processor. This allows the cluster to detect processors that are shutting down and smoothly reassign their processes.

This change adds a new layer of directories when creating processors. Process documents are no longer stored directly under the process directory, instead a precesses subdirectory holds those documents and a processors directory holds the documents representing each active processor in the cluster.

}
} catch( InterruptedException e ) {
logger.warn("Interrupted while shutting down processes.");
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This block should be wrapped up and moved out of the way.

@ctrimble
Copy link
Contributor Author

Cleanup task should cleanup processor documents.

@ctrimble
Copy link
Contributor Author

Fixes #5

@ctrimble ctrimble force-pushed the feature_assignment-index-tracking branch from ef3a807 to a067f25 Compare October 21, 2015 16:34
ctrimble added a commit that referenced this pull request Oct 21, 2015
@ctrimble ctrimble merged commit 227cb03 into develop Oct 21, 2015
@ctrimble ctrimble deleted the feature_assignment-index-tracking branch October 21, 2015 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant