search

coiled / feedback

A place to provide Coiled feedback
14 stars 3 forks source link

Cluster.get_client() method #144

Open gjoseph92 opened 3 years ago

gjoseph92 commented 3 years ago

Dask Gateway clusters have a convenient get_client() method. Tiny detail, but I find this slightly more ergonomic than import distributed; client = distributed.Client(coiled_cluster)—less typing, and you don't have to import distributed yourself if you don't need it. Would be nice if coiled.Cluster had this too.

jrbourbeau commented 3 years ago

Thanks @gjoseph92! Just wanted to cross-reference a distributed PR to upstream get_client() from dask-gateway into distributed https://github.com/dask/distributed/pull/4022. There was some pushback at the time to adding a get_client() method, though maybe opinions have changed since then. FWIW I would be in favor of coiled doing whatever distributed ends up doing.