-
Notifications
You must be signed in to change notification settings - Fork 3
Add async module #73
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
base: main
Are you sure you want to change the base?
Add async module #73
Conversation
jonathonmcmurray
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
few additional things needed here
di/async/async.q
Outdated
| / error trapping sending the query down the handle followed by an async flush | ||
| .[{x@y; x(::);1b};(h;tosend);0b]} | ||
|
|
||
| broadcastdeferred:{[handles;query] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rename this to just deferred seeing as we don't have a non-broadcast version any more
di/async/init.q
Outdated
| @@ -0,0 +1,3 @@ | |||
| \l ::async.q | |||
|
|
|||
| export:([deferred;postback;broadcastdeferred;broadcastpostback]) No newline at end of file | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this needs updated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tests need updated, they still reference both broadcast & non-broadcast versions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
docs need updated, still reference both broadcast & non-broadcast versions
di/async/async.md
Outdated
| @@ -0,0 +1,69 @@ | |||
| ## KDB+/Q Asynchronous Communication Library | |||
|
|
|||
| This library provides kdb+/q functions for sending either deferred synchronous or asynchronous postback requests from a client process over a handle or list of handles, with error trapping at various points. Either type of request can be sent via conventional kdb+/q IPC or asynchronous broadcast. | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| This library provides kdb+/q functions for sending either deferred synchronous or asynchronous postback requests from a client process over a handle or list of handles, with error trapping at various points. Either type of request can be sent via conventional kdb+/q IPC or asynchronous broadcast. | |
| This library provides kdb+/q functions for sending either deferred synchronous or asynchronous postback requests from a client process over a handle or list of handles, with error trapping at various points. Either type of request can be sent via conventional kdb+/q IPC or asynchronous broadcast. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(also last sentence needs updated)
di/async/async.q
Outdated
| @@ -0,0 +1,25 @@ | |||
| / library for sending async messages from a client process | |||
|
|
|||
| send:{[w;h;q] | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
send function can be removed also as only relates to non-broadcast versions
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.