-
Notifications
You must be signed in to change notification settings - Fork 76
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
Pipeline runs as background jobs #154
Comments
Can we integrate approach 1 with some sort of notification functionality?
Please let me know if I am missing something crucial or going wrong somewhere. |
Hey @nagxsan thanks for your input and sorry for the delayed response!
Agreed, that's likely.
This issue is about the API requests from code. See the API reference. Currently, the pipeline run requests are simply a REST request-response model. The best (and maybe the only) way I can imagine us sending a notification to the code SDKs is exactly approach 2 I suggested. Do you have another idea on how to send a notification? |
I was thinking approach 1 in that situation sounds the most straightforward:
Can you please tell me why is this the case? The pipeline runs are anyways long runs which the user will probably just submit the job and forget about it for a while. Even if they manually refresh after sometime, the |
Currently, our network configurations in the managed versions cut TCP (TLS to be more specific) connections after 350 seconds. For most of our APIs this is more than enough, but pipeline runs sometimes take longer, especially with the rise of larger/slower models, like o1.
We need to add an ability to run pipelines as background jobs.
Currently, this is rather a discussion, not a call for PR. I see two possible ways forward, but we are open to more suggestions as always.
run_id
and polls on it.We are open to discussions for the best way forward and any other suggestions
The text was updated successfully, but these errors were encountered: