[jira] [Commented] (MAPREDUCE-4296) remove old aggregated logs

2012-08-17 Thread Robert Joseph Evans (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-4296?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13436999#comment-13436999 ] Robert Joseph Evans commented on MAPREDUCE-4296: Will do. I'll remove

[jira] [Commented] (MAPREDUCE-4296) remove old aggregated logs

2012-08-17 Thread Thomas Graves (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-4296?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13437028#comment-13437028 ] Thomas Graves commented on MAPREDUCE-4296: -- +1 Thanks Bobby!

[jira] [Commented] (MAPREDUCE-4296) remove old aggregated logs

2012-08-17 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-4296?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13437030#comment-13437030 ] Hadoop QA commented on MAPREDUCE-4296: -- +1 overall. Here are the results of

[jira] [Commented] (MAPREDUCE-4296) remove old aggregated logs

2012-08-16 Thread Thomas Graves (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-4296?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13436167#comment-13436167 ] Thomas Graves commented on MAPREDUCE-4296: -- Mostly looks good. couple minor

[jira] [Commented] (MAPREDUCE-4296) remove old aggregated logs

2012-08-15 Thread Robert Joseph Evans (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-4296?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13435122#comment-13435122 ] Robert Joseph Evans commented on MAPREDUCE-4296: Thanks for the review

[jira] [Commented] (MAPREDUCE-4296) remove old aggregated logs

2012-08-15 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-4296?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13435180#comment-13435180 ] Hadoop QA commented on MAPREDUCE-4296: -- +1 overall. Here are the results of

[jira] [Commented] (MAPREDUCE-4296) remove old aggregated logs

2012-08-15 Thread Jason Lowe (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-4296?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13435262#comment-13435262 ] Jason Lowe commented on MAPREDUCE-4296: --- Getting close, a few more points: *

[jira] [Commented] (MAPREDUCE-4296) remove old aggregated logs

2012-08-15 Thread Robert Joseph Evans (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-4296?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13435279#comment-13435279 ] Robert Joseph Evans commented on MAPREDUCE-4296: Yes I agree, it was

[jira] [Commented] (MAPREDUCE-4296) remove old aggregated logs

2012-08-15 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-4296?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13435326#comment-13435326 ] Hadoop QA commented on MAPREDUCE-4296: -- +1 overall. Here are the results of

[jira] [Commented] (MAPREDUCE-4296) remove old aggregated logs

2012-08-15 Thread Jason Lowe (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-4296?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13435367#comment-13435367 ] Jason Lowe commented on MAPREDUCE-4296: --- +1 (non-binding), lgtm. Thanks Bobby!

[jira] [Commented] (MAPREDUCE-4296) remove old aggregated logs

2012-08-14 Thread Jason Lowe (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-4296?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13434587#comment-13434587 ] Jason Lowe commented on MAPREDUCE-4296: --- Patch looks good overall, some cleanup

[jira] [Commented] (MAPREDUCE-4296) remove old aggregated logs

2012-08-14 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-4296?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13434659#comment-13434659 ] Hadoop QA commented on MAPREDUCE-4296: -- +1 overall. Here are the results of