snuspl / cruise

Cruise: A Distributed Machine Learning Framework with Automatic System Configuration
Apache License 2.0
26 stars 2 forks source link

[CAY-1268] Introduce Tasklet #1269

Closed wynot12 closed 6 years ago

wynot12 commented 6 years ago

Resolves #1268 Resolves cmssnu/elastic-tables#15 Resolves cmssnu/elastic-tables#120 Resolves cmssnu/elastic-tables#133

This PR

Additionally this PR resolves #1225. Introducing Tasklet requires clean separation of each tasklet instance in executors. And this separation resolves #1225 consequently.

Note that this PR does not include changing dolphin-jobserver to share executors across job.

wynot12 commented 6 years ago

@yunseong I'd like to give you a brief overview of the PR. Ping me when you have time :)

wynot12 commented 6 years ago

I'll add inline comments until tomorrow.

yunseong commented 6 years ago

Thanks for the work! Let me review it this afternoon.

wynot12 commented 6 years ago

I've just added comments :)

wynot12 commented 6 years ago

I've resolved your comments! thx :)