Skip to content

wait_for_endpoint should have a timeout value #2539

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

Closed
larroy opened this issue Jul 28, 2021 · 1 comment
Closed

wait_for_endpoint should have a timeout value #2539

larroy opened this issue Jul 28, 2021 · 1 comment

Comments

@larroy
Copy link
Contributor

larroy commented Jul 28, 2021

Describe the feature you'd like

wait_for_endpoint can potentially hang up forever, this is rare but we have observed such case where the endpoint fails to come online.

https://sagemaker.readthedocs.io/en/stable/api/utility/session.html#sagemaker.session.Session.wait_for_endpoint

we would like to have wait_for_endpoint have a timeout value so it will return control to the program after a specified time interval.

How would this feature be used? Please describe.
as above

Describe alternatives you've considered
Using a signal handler to interrupt the wait? but it's clunky and not elegant.

Additional context
Add any other context or screenshots about the feature request here.

@akrishna1995
Copy link
Contributor

Hi @larroy - Thanks for raising a PR for this feature request. Feel free to reopen the PR and our team can help review and merge it. Meanwhile I have added this feature request to our backlog of feature requests and may consider putting it into future SDK versions. I will go ahead and close the issue now, please let me know if you have any more feedback.

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

No branches or pull requests

3 participants