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

wip: xxx: introduce a way to submit and listen isolated calls #103

Closed
wants to merge 2 commits into from

Conversation

isidentical
Copy link
Collaborator

The idea underneath is, currently for every isolate run, you basically need two peers (a client and a server) which makes both sides hold state without a good reason. For the world of HTTP servers, we can essentially fire-and-forget (which is not a good name for this stuff where we actually care about the status of the task) them and let the gateway itself hold as little state/connections as possible.

For achieving this goal quickly, this PR proposes a Submit endpoint in addition to the existing Run endpoint in the isolate server where you can fire a task and pass a callback which will be used to communicate the logs and the results back. This is still a bit fragile (especially without any sort of fault tolarance on when the callback URL fails) and might be even slower (plain HTTP 🤢) but all of these properties can be improved upon once we are confident in the architecture. The main goal here is to provide the most basic form of fire-and-forget mechanism to isolate where we can start using and experimenting with it on the server and eventually make it nicer.

@efiop efiop force-pushed the submit-and-listen branch from 6c86152 to 573d2fe Compare March 30, 2024 22:54
@efiop efiop self-assigned this Mar 30, 2024
@efiop efiop force-pushed the submit-and-listen branch 2 times, most recently from 1159d69 to edf044e Compare April 3, 2024 15:53
@kakkoyun
Copy link
Contributor

kakkoyun commented Apr 3, 2024

/subscribe

@efiop efiop force-pushed the submit-and-listen branch 4 times, most recently from 034a79b to 25801bb Compare April 3, 2024 21:33
@efiop efiop force-pushed the submit-and-listen branch from 25801bb to 37b099b Compare May 26, 2024 22:23
@efiop efiop force-pushed the submit-and-listen branch from 37b099b to 8eb214d Compare May 27, 2024 12:28
@efiop
Copy link
Contributor

efiop commented May 27, 2024

Closing to continue in a separate PR (don't want to butcher content here).

@efiop efiop closed this May 27, 2024
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

Successfully merging this pull request may close these issues.

3 participants