Vinod Kumar Vavilapalli commented on YARN-1368:

bq. Unless I'm missing something, Anubhav was working on this JIRA. It is great 
that Jian did the refactoring to have common code for the schedulers and some 
testcases for it, but most of the work has been done by Anubhav and he was 
working actively on it. We should reassign the JIRA back to Anubhav and let him 
drive it to completion, agree?
[~tucu00], if you see the history of this ticket, there was nothing here that 
showed that Anubhav was working on this. There was a prototype patch on the 
umbrella JIRA, but the part of the patch that focuses on this JIRA was 
completely off the mark. It is specifically for these reasons that I publicly 
discouraged posting prototype patch for this well-defined feature. In this 
case, the prototype was off the mark, and folks(not just [~jianhe]) were 
confused about the individual tasks filed.

bq. Jian He, I understand the patch is taking a different approach, which is 
based on the work Anubhav started. Instead hijacking the JIRA, the correct way 
should have been proposing to the assignee/author of the original patch the 
changes and offering to contribute/breakdown tasks. Please do so next time.
[~tucu00], I just looked the prototype as well as this patch. Not only is this 
patch taking a different approach, it is also *not* based on the work in the 
prototype. The approach in the prototype is pretty broken - everyone's free to 
look back on both the patches and argue about this - this one aims at 
correcting it by starting from scratch and moving beyond to set the right stage 
for the entire effort.

Even if it were, it is sometimes a major burden to correct original patches, 
specifically when their approaches are completely botched, and then direct them 
towards the right approach. This one's one of those.

I agree in general that he shouldn't have re-purposed this JIRA. This JIRA's 
original purpose was a very small subset of what it is doing now.

> Common work to re-populate containers’ state into scheduler
> -----------------------------------------------------------
>                 Key: YARN-1368
>                 URL: https://issues.apache.org/jira/browse/YARN-1368
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Bikas Saha
>            Assignee: Jian He
>         Attachments: YARN-1368.1.patch, YARN-1368.2.patch, YARN-1368.3.patch, 
> YARN-1368.4.patch, YARN-1368.5.patch, YARN-1368.combined.001.patch, 
> YARN-1368.preliminary.patch
> YARN-1367 adds support for the NM to tell the RM about all currently running 
> containers upon registration. The RM needs to send this information to the 
> schedulers along with the NODE_ADDED_EVENT so that the schedulers can recover 
> the current allocation state of the cluster.

This message was sent by Atlassian JIRA

Reply via email to