-
-
Notifications
You must be signed in to change notification settings - Fork 6
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
Proof of concept: Synchronous ds <-> dstopic IPC #14
Comments
cc @ysz are you still interested in giving this a shot and coming back to this where you left off? |
@zkat isn't dstopic archived? |
@daubaris it lives in this repo now! |
@zkat Oh, got it. I would like to tackle it (if possible of course (and would need to familiarize myself with the codebase))! |
Please go ahead! That would be really awesome! Let me know if you have any questions, it's a bit of a hard problem |
@zkat I was thinking about creating a separate In addition, |
One big challenge for the ds/dstopic divide is this idea that there will be a single, long-lived dstopic binary that ds will then communicate with to request data be warmed up. But this comes with a tricky bit: There’s no easy way to do synchronous IPC in Node, and require is a synchronous operation.
So here’s a challenge: We need at -least- a proof of concept of how to send requests from ds to dstopic synchronously. This includes a way to wait for dstopic to be done with its task and send a success response back before the call is over.
Hint: one way to possibly do this is by using fs.writeSync/fs.readSync with a local socket of some sort. I forget the details, but that’s the first avenue I’d explore to accomplish this.
(original thread)
The text was updated successfully, but these errors were encountered: