[jira] [Commented] (TEZ-1911) MergeManager's unconditionalReserve() should check for memory limits before allocating memory to IntermediateMemoryToMemoryMerger

2016-02-26 Thread Rajesh Balamohan (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1911?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15170247#comment-15170247 ] Rajesh Balamohan commented on TEZ-1911: --- committed addendum patch to master. >>> comm

[jira] [Commented] (TEZ-1911) MergeManager's unconditionalReserve() should check for memory limits before allocating memory to IntermediateMemoryToMemoryMerger

2016-02-26 Thread Jason Lowe (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1911?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15169646#comment-15169646 ] Jason Lowe commented on TEZ-1911: - I'll hold off on the cherry-pick to branch-0.7 until the

[jira] [Commented] (TEZ-1911) MergeManager's unconditionalReserve() should check for memory limits before allocating memory to IntermediateMemoryToMemoryMerger

2016-02-26 Thread Siddharth Seth (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1911?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15169536#comment-15169536 ] Siddharth Seth commented on TEZ-1911: - According to findbugs {code} Unsynchronized acces

[jira] [Commented] (TEZ-1911) MergeManager's unconditionalReserve() should check for memory limits before allocating memory to IntermediateMemoryToMemoryMerger

2016-02-26 Thread Siddharth Seth (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1911?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15169442#comment-15169442 ] Siddharth Seth commented on TEZ-1911: - Please go ahead. > MergeManager's unconditionalR

[jira] [Commented] (TEZ-1911) MergeManager's unconditionalReserve() should check for memory limits before allocating memory to IntermediateMemoryToMemoryMerger

2016-02-26 Thread Jason Lowe (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1911?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15169231#comment-15169231 ] Jason Lowe commented on TEZ-1911: - This picks mostly clean to branch-0.7 except for a minor

[jira] [Commented] (TEZ-1911) MergeManager's unconditionalReserve() should check for memory limits before allocating memory to IntermediateMemoryToMemoryMerger

2016-02-25 Thread TezQA (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1911?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15168469#comment-15168469 ] TezQA commented on TEZ-1911: {color:red}-1 overall{color}. Here are the results of testing the

[jira] [Commented] (TEZ-1911) MergeManager's unconditionalReserve() should check for memory limits before allocating memory to IntermediateMemoryToMemoryMerger

2016-02-25 Thread Siddharth Seth (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1911?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15168468#comment-15168468 ] Siddharth Seth commented on TEZ-1911: - +1. Thanks! > MergeManager's unconditionalReserv

[jira] [Commented] (TEZ-1911) MergeManager's unconditionalReserve() should check for memory limits before allocating memory to IntermediateMemoryToMemoryMerger

2016-02-25 Thread Rajesh Balamohan (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1911?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15168327#comment-15168327 ] Rajesh Balamohan commented on TEZ-1911: --- Agreed. Added a check for early-exit before a

[jira] [Commented] (TEZ-1911) MergeManager's unconditionalReserve() should check for memory limits before allocating memory to IntermediateMemoryToMemoryMerger

2016-02-25 Thread TezQA (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1911?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15168223#comment-15168223 ] TezQA commented on TEZ-1911: {color:red}-1 overall{color}. Here are the results of testing the

[jira] [Commented] (TEZ-1911) MergeManager's unconditionalReserve() should check for memory limits before allocating memory to IntermediateMemoryToMemoryMerger

2016-02-25 Thread Siddharth Seth (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1911?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15168127#comment-15168127 ] Siddharth Seth commented on TEZ-1911: - bq. One more question - does the interrupt status

[jira] [Commented] (TEZ-1911) MergeManager's unconditionalReserve() should check for memory limits before allocating memory to IntermediateMemoryToMemoryMerger

2016-02-25 Thread Siddharth Seth (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1911?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15166911#comment-15166911 ] Siddharth Seth commented on TEZ-1911: - One more question - does the interrupt status hav

[jira] [Commented] (TEZ-1911) MergeManager's unconditionalReserve() should check for memory limits before allocating memory to IntermediateMemoryToMemoryMerger

2016-02-25 Thread Siddharth Seth (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1911?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15166910#comment-15166910 ] Siddharth Seth commented on TEZ-1911: - {code} LOG.info(inputContext.getSourceVertexName(

[jira] [Commented] (TEZ-1911) MergeManager's unconditionalReserve() should check for memory limits before allocating memory to IntermediateMemoryToMemoryMerger

2016-02-17 Thread TezQA (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1911?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15151700#comment-15151700 ] TezQA commented on TEZ-1911: {color:red}-1 overall{color}. Here are the results of testing the