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

Better cluster manager, Pre-instantiation fix, and better monitoring #64

Merged
merged 25 commits into from
Oct 9, 2012

Conversation

jimfcarroll
Copy link
Collaborator

This is now ready and contains these major changes:

  1. Refactor of the Cluster manager
  2. (1) results in the need to refactor the Routing Strategy
  3. Fix for the race condition between the Routing Strategy and Pre-instantiation in issue Possible race conditions during pre-instantiation (micro-shard allocation and mp instantiation) #60
  4. Added sent/recieved byte count metrics. Closes issue Add messages byte count to the stats #63.
  5. Separately track mp failures and dempsy failures during message processing. Closes issue Account for Message Failed due to Framework or message processor #62.
  6. Add the ability to measure the mp message handling time. This closes issue Collect stats on MP latency #61
  7. The monitoring for Adaptors in a distributed application should have a prefix that's consistent with the monitors for non-Adaptors in the same application. This closes issue adaptor showing up under graphite - under local folder #66.

Jim Carroll added 20 commits September 27, 2012 11:54
…in preparation for an alternate micro-sharding based strategy.
…Strategy initialization durring the PreInstantiation phase.
… ability to measure the mp message handling time. This closes issue Dempsy#61
…, to an implementation specific interface. This cleans up the publicly facing API a bit.
…executing simultaneously. Successive calls now end currently executing calls and then proceed.
@jimfcarroll jimfcarroll mentioned this pull request Oct 6, 2012
jimfcarroll pushed a commit that referenced this pull request Oct 9, 2012
Better cluster manager, Pre-instantiation fix, and better monitoring
@jimfcarroll jimfcarroll merged commit 01c20af into Dempsy:master Oct 9, 2012
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.

2 participants