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

consider event-based flux_rpc_multi() #681

Closed
garlick opened this issue May 25, 2016 · 2 comments
Closed

consider event-based flux_rpc_multi() #681

garlick opened this issue May 25, 2016 · 2 comments

Comments

@garlick
Copy link
Member

garlick commented May 25, 2016

flux_rpc_multi() currently sends a request to each targetted rank, which the broker routes over the ring network. This phase of the RPC could be replaced with a single event.

Responses could then be routed back to rank 0 over the TBON, and then routed to requestor.

Some design is required to ensure messages are routed and dispatched properly.

@garlick
Copy link
Member Author

garlick commented Aug 11, 2016

After #689, flux_rpc_multi() routes requests over the TBON, which improved latency dramatically. This solved observed performance problems with module loading.

There is still an opportunity to reduce the number of requests from N to 1 through the use of events, so leaving this open.

@garlick
Copy link
Member Author

garlick commented Dec 30, 2016

Closing this, as it seems like merely grafting events into the flux_rpc_multi() falls somewhere between a premature optimization and an insufficiently ambitious design for a pattern bordering on the "collectives" space

@garlick garlick closed this as completed Dec 30, 2016
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

No branches or pull requests

1 participant