[jira] [Commented] (MAPREDUCE-2905) CapBasedLoadManager incorrectly allows assignment when assignMultiple is true (was: assignmultiple per job)

2011-11-22 Thread Matei Zaharia (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-2905?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13155358#comment-13155358 ] Matei Zaharia commented on MAPREDUCE-2905: -- The patch looks good, except that

[jira] [Commented] (MAPREDUCE-2905) CapBasedLoadManager incorrectly allows assignment when assignMultiple is true (was: assignmultiple per job)

2011-11-16 Thread Todd Lipcon (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-2905?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13151533#comment-13151533 ] Todd Lipcon commented on MAPREDUCE-2905: So, I disagree with some of what you

[jira] [Commented] (MAPREDUCE-2905) CapBasedLoadManager incorrectly allows assignment when assignMultiple is true (was: assignmultiple per job)

2011-11-15 Thread Matei Zaharia (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-2905?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13150690#comment-13150690 ] Matei Zaharia commented on MAPREDUCE-2905: -- Yup, the patch itself also looks

[jira] [Commented] (MAPREDUCE-2905) CapBasedLoadManager incorrectly allows assignment when assignMultiple is true (was: assignmultiple per job)

2011-11-15 Thread Todd Lipcon (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-2905?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13150706#comment-13150706 ] Todd Lipcon commented on MAPREDUCE-2905: TestFairScheduler is failing with

[jira] [Commented] (MAPREDUCE-2905) CapBasedLoadManager incorrectly allows assignment when assignMultiple is true (was: assignmultiple per job)

2011-11-15 Thread Jeff Bean (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-2905?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13150716#comment-13150716 ] Jeff Bean commented on MAPREDUCE-2905: -- More info about the failure?

[jira] [Commented] (MAPREDUCE-2905) CapBasedLoadManager incorrectly allows assignment when assignMultiple is true (was: assignmultiple per job)

2011-11-15 Thread Jeff Bean (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-2905?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13150964#comment-13150964 ] Jeff Bean commented on MAPREDUCE-2905: -- testFairScheduler is buggy. In one

[jira] [Commented] (MAPREDUCE-2905) CapBasedLoadManager incorrectly allows assignment when assignMultiple is true (was: assignmultiple per job)

2011-11-15 Thread Jeff Bean (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-2905?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13150968#comment-13150968 ] Jeff Bean commented on MAPREDUCE-2905: -- (strike the last incomplete sentence of

[jira] [Commented] (MAPREDUCE-2905) CapBasedLoadManager incorrectly allows assignment when assignMultiple is true (was: assignmultiple per job)

2011-11-15 Thread Jeff Bean (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-2905?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13150969#comment-13150969 ] Jeff Bean commented on MAPREDUCE-2905: -- trying to think about how to fix the

[jira] [Commented] (MAPREDUCE-2905) CapBasedLoadManager incorrectly allows assignment when assignMultiple is true (was: assignmultiple per job)

2011-11-15 Thread Todd Lipcon (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-2905?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13150994#comment-13150994 ] Todd Lipcon commented on MAPREDUCE-2905: Hey Jeff - hope you didnt spend much

[jira] [Commented] (MAPREDUCE-2905) CapBasedLoadManager incorrectly allows assignment when assignMultiple is true (was: assignmultiple per job)

2011-11-15 Thread Jeff Bean (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-2905?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13151008#comment-13151008 ] Jeff Bean commented on MAPREDUCE-2905: -- just a bunch of tuesday. no big deal.

[jira] [Commented] (MAPREDUCE-2905) CapBasedLoadManager incorrectly allows assignment when assignMultiple is true (was: assignmultiple per job)

2011-11-14 Thread Eli Collins (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-2905?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13150050#comment-13150050 ] Eli Collins commented on MAPREDUCE-2905: Hi Matei, Are you +1 on Todd's patch

[jira] [Commented] (MAPREDUCE-2905) CapBasedLoadManager incorrectly allows assignment when assignMultiple is true (was: assignmultiple per job)

2011-10-27 Thread Matei Zaharia (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-2905?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13137787#comment-13137787 ] Matei Zaharia commented on MAPREDUCE-2905: -- Sorry for taking a bit of time to

[jira] [Commented] (MAPREDUCE-2905) CapBasedLoadManager incorrectly allows assignment when assignMultiple is true (was: assignmultiple per job)

2011-10-17 Thread Todd Lipcon (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-2905?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13129451#comment-13129451 ] Todd Lipcon commented on MAPREDUCE-2905: The patch seems to reformat a bunch

[jira] [Commented] (MAPREDUCE-2905) CapBasedLoadManager incorrectly allows assignment when assignMultiple is true (was: assignmultiple per job)

2011-10-17 Thread Jeff Bean (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-2905?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13129453#comment-13129453 ] Jeff Bean commented on MAPREDUCE-2905: -- Oh I'm happy to do it. Is two-space

[jira] [Commented] (MAPREDUCE-2905) CapBasedLoadManager incorrectly allows assignment when assignMultiple is true (was: assignmultiple per job)

2011-10-12 Thread Harsh J (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-2905?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13125843#comment-13125843 ] Harsh J commented on MAPREDUCE-2905: Jeff, I'll leave the final review to people

[jira] [Commented] (MAPREDUCE-2905) CapBasedLoadManager incorrectly allows assignment when assignMultiple is true (was: assignmultiple per job)

2011-10-12 Thread Jeff Bean (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-2905?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13125904#comment-13125904 ] Jeff Bean commented on MAPREDUCE-2905: -- The test is a sleep job with slightly

[jira] [Commented] (MAPREDUCE-2905) CapBasedLoadManager incorrectly allows assignment when assignMultiple is true (was: assignmultiple per job)

2011-10-12 Thread Todd Lipcon (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-2905?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13125963#comment-13125963 ] Todd Lipcon commented on MAPREDUCE-2905: We do need a junit test. Manual tests

[jira] [Commented] (MAPREDUCE-2905) CapBasedLoadManager incorrectly allows assignment when assignMultiple is true (was: assignmultiple per job)

2011-10-12 Thread Jeff Bean (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-2905?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13126275#comment-13126275 ] Jeff Bean commented on MAPREDUCE-2905: -- I don't understand why the existing JUnit

[jira] [Commented] (MAPREDUCE-2905) CapBasedLoadManager incorrectly allows assignment when assignMultiple is true (was: assignmultiple per job)

2011-10-12 Thread Todd Lipcon (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-2905?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13126350#comment-13126350 ] Todd Lipcon commented on MAPREDUCE-2905: That's exactly the point. If the

[jira] [Commented] (MAPREDUCE-2905) CapBasedLoadManager incorrectly allows assignment when assignMultiple is true (was: assignmultiple per job)

2011-10-12 Thread Todd Lipcon (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-2905?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13126358#comment-13126358 ] Todd Lipcon commented on MAPREDUCE-2905: The point isn't to test that this

[jira] [Commented] (MAPREDUCE-2905) CapBasedLoadManager incorrectly allows assignment when assignMultiple is true (was: assignmultiple per job)

2011-10-10 Thread Jeff Bean (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-2905?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13124603#comment-13124603 ] Jeff Bean commented on MAPREDUCE-2905: -- Verified that this fix works. Can someone