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

Monovertex changes to support serving as a builtin source #2350

Open
Tracked by #2244
yhl25 opened this issue Jan 22, 2025 · 0 comments · May be fixed by #2382
Open
Tracked by #2244

Monovertex changes to support serving as a builtin source #2350

yhl25 opened this issue Jan 22, 2025 · 0 comments · May be fixed by #2382
Assignees
Milestone

Comments

@yhl25
Copy link
Contributor

yhl25 commented Jan 22, 2025

Some changes to the user facing APIs also will be made in this PR:

  • POST /v1/process/sync - Receive payload and return result synchronously. If client gets disconnects (network error or timeouts), the results can still be retrieved using /v1/process/serve endpoint
  • POST /v1/process/async - Receive payload and process asynchronously.
  • GET /v1/process/serve - Receive processing status and/or results. The request may be submitted using /sync or /async endpoint.
  • GET /v1/process/message - Retrive message graph for a request id

When submitting the request (either /sync or /async), the request id must not already exist in the store (for the duration of the store TTL). This is needed to avoid accidental overwrite of the results. A query parameter can be provided to allow overwriting.

The callback info will be saved in memory. It will be saved to Redis when the pod is shutting down or when all callbacks for a request id is received.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants