search

taskgraph / mr

1 stars 1 forks source link

mapreduce new #1

Open plutoshe opened 9 years ago

xiang90 commented 9 years ago

Is this ready for review?

hongchaodeng commented 9 years ago

I would like to see the design doc to be agreed upon and finished completely first:

https://docs.google.com/document/d/15pT8tzvGbuxlPEKB9901D26da2_wEn-zr3OpylQqDE4/edit?usp=sharing

I think the lack of consensus on arch design impedes development so far, especially when there are a lot of new change.

plutoshe commented 9 years ago

Not yet, I haven't finished the main code.

xiaoyunwu commented 9 years ago

@Hongchao,

Mapreduce is an application mostly, its interface to user is fairly simple. So it is ok for us to first implement something that works, and then worry about how to improve the implementation.

That said, @Pluto, please update design doc so that it is easy for other to get a high level understanding of what we try to achieve.

Xiaoyun

On Tue, Jun 2, 2015 at 1:45 PM, PlutoShe notifications@github.com wrote:

Not yet, I haven't finished the main code.

— Reply to this email directly or view it on GitHub https://github.com/taskgraph/mr/pull/1#issuecomment-107814630.

xiang90 commented 9 years ago

Let me know when this is ready for a review... the pr is getting bigger...

plutoshe commented 9 years ago

Sure, when it finish, I will notify in mail