Add async module #73
Open
Add async module #73
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| / for sending asynchronous postback message to a list of handles where the message is wrapped in the function postback | ||
| send[0b;;({[q;p] (p;@[value;q;{"error: server fail: ",x}])};query;postback)] each handles:neg abs handles,()} | ||
|
|
||
| broadcastdeferred:{[handles;query] |
Member
There was a problem hiding this comment.
is there ever a reason not to use broadcast? i.e. should we just get rid of the non-broadcast versions and rename the broadcast ones?
Author
There was a problem hiding this comment.
Only scenario I can think of is if there's an issue with any of the handles - with broadcast, one invalid handle causes none of the messages to be sent, whereas for the non-broadcast versions only the message to the invalid handle will fail. Bit of a rare case though, and ideally some connection management code would prevent that situation in the first place. So to tidy things up, it's probably not worth keeping the non-broadcast versions in that case
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Utility functions to send deferred sync and async postback IPC messages with error trapping at various points.
Also includes function variants to send these via async broadcast.