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

Improve task polling #184

Open
tdstein opened this issue Apr 26, 2024 · 1 comment
Open

Improve task polling #184

tdstein opened this issue Apr 26, 2024 · 1 comment
Labels
enhancement New feature or request sdk Used for automation

Comments

@tdstein
Copy link
Collaborator

tdstein commented Apr 26, 2024

The existing task polling is naive and doesn't consider the various edge cases around output length, polling algorithms, etc...

Modify the implementation to take advantage of the log appending support in the API.
Modify the polling algorithm to reduce server resource consumption (i.e., do polling client side instead of server side).
Check if there is a need for supporting more advanced polling algorithms (.e.g, backoff policies, etc).

@tdstein tdstein added the enhancement New feature or request label Apr 26, 2024
@tdstein tdstein added this to the 0.4.0 milestone Apr 26, 2024
@nealrichardson
Copy link
Collaborator

Another thought, following the observation on #183 (comment): Request with some reasonable wait amount for the first request, since if it is a quick task, it will return early. Then if longer than that, start polling.

@tdstein tdstein removed this from the 0.4.0 milestone Sep 4, 2024
@tdstein tdstein added the sdk Used for automation label Sep 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request sdk Used for automation
Projects
None yet
Development

No branches or pull requests

2 participants