dask / old-dask-yarn

Deprecated, please use https://github.com/jcrist/skein or https://github.com/dask/dask-yarn instead
7 stars 2 forks source link

feature/add_remove_workers #4

Closed quasiben closed 7 years ago

quasiben commented 7 years ago

Tied to PR https://github.com/dask/knit/pull/61

quasiben commented 7 years ago

@mrocklin when you have some time can you look this over ?

mrocklin commented 7 years ago

Ah, yes sorry. I did give this a shot and everything work smoothly for me. Thanks as always for maintaining this. It's a great help. I think I forgot to mention that things were working well here only because I became too distracted by the results that I was seeing on the experiment I was running :)

quasiben commented 7 years ago

Is it possible you think you are commenting on https://github.com/dask/dask-ec2/pull/76 ?

mrocklin commented 7 years ago

ah yes, indeed. My mistake

On Mon, Mar 27, 2017 at 10:03 PM, Benjamin Zaitlen <notifications@github.com

wrote:

Is it possible you think you are commenting on dask/dask-ec2#76 https://github.com/dask/dask-ec2/pull/76 ?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/dask/dask-yarn/pull/4#issuecomment-289640325, or mute the thread https://github.com/notifications/unsubscribe-auth/AASszKOXxgoGd9P2s7jmG8WrDpNFbULJks5rqGp_gaJpZM4LbJDu .