-
Notifications
You must be signed in to change notification settings - Fork 98
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
Document guarantees provided by Management API #241
Comments
There are no ordering guarantees between Agents, no. The event bus is serial and single threaded, so we should have some guarantees, but not many. Anything that is published by the NC will be treated in FIFO order. There are no ordering guarantees pertaining to entries published to the event bus by other processes or agents though. Can messages be lost? Locally, no, I don't think so. Unless the forkIO thread on which the event bus relies is killed, in which case we've got bigger problems anyway. Perhaps all the event state should be isolated further? This is similar to the problem of "what happens if the NC thread terminates?" I think. Could messages be reordered: No, I don't see how they could. |
@qnikst - how do you want this documented, and where? I'd like to close this down so that we can move on with blocked PRs... |
@qnikst ... ping! I'd like to know where you want these things documented, and I'll provide a PR today or tomorrow. I'd like to get on with #265 and #184 so we can look at a minor/patch release, so I can stabilise the version of CH on which various other libraries rely (i.e., to resolve the discussion in haskell-distributed/distributed-process-extras#6 (comment)). More importantly, do you agree with all of the following assertions about this....
I can point to the code fragments which I believe back this up, if that's useful... |
I think I agree with all of the points, and current implementation match those. |
Okay awesome, I'll add those points to the docs for Debug/Management and submit a PR. |
It worth documenting what are the guarantees that management API provides:
The text was updated successfully, but these errors were encountered: