search

dask / knit

Deprecated, please use https://github.com/jcrist/skein or https://github.com/dask/dask-yarn instead
http://knit.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
53 stars 10 forks source link

Add timeouts to all REST calls in YARNAPI #90

Closed martindurant closed 6 years ago

martindurant commented 6 years ago

Fixes #85 (at least, you'll get a realistic exception rather than hanging).

martindurant commented 6 years ago

@jcrist , comments on this change?

jcrist commented 6 years ago

At least this won't hang anymore. Not sure if there's a better solution, but for now I can just catch the exception on exit.