[jira] [Commented] (YARN-3324) TestDockerContainerExecutor should clean test docker image from local repository after test is done

2015-07-28 Thread Chen He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3324?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14644693#comment-14644693 ] Chen He commented on YARN-3324: --- IMHO, we can test whether the docker image is used (by any

[jira] [Commented] (YARN-3324) TestDockerContainerExecutor should clean test docker image from local repository after test is done

2015-07-28 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3324?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14644758#comment-14644758 ] Hadoop QA commented on YARN-3324: - \\ \\ | (x) *{color:red}-1 overall{color}* | \\ \\ ||

[jira] [Commented] (YARN-3324) TestDockerContainerExecutor should clean test docker image from local repository after test is done

2015-03-28 Thread Chen He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3324?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14385158#comment-14385158 ] Chen He commented on YARN-3324: --- {quote}1. There will be overhead of downloading the image

[jira] [Commented] (YARN-3324) TestDockerContainerExecutor should clean test docker image from local repository after test is done

2015-03-28 Thread Chen He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3324?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14385164#comment-14385164 ] Chen He commented on YARN-3324: --- If we decide not to delete test images, we can just create

[jira] [Commented] (YARN-3324) TestDockerContainerExecutor should clean test docker image from local repository after test is done

2015-03-28 Thread Ravindra Kumar Naik (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3324?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14385160#comment-14385160 ] Ravindra Kumar Naik commented on YARN-3324: --- {quote}The overhead of downloading

[jira] [Commented] (YARN-3324) TestDockerContainerExecutor should clean test docker image from local repository after test is done

2015-03-27 Thread Ravi Prakash (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3324?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14384274#comment-14384274 ] Ravi Prakash commented on YARN-3324: Hi Ravindra! Is there a reason you want to delete

[jira] [Commented] (YARN-3324) TestDockerContainerExecutor should clean test docker image from local repository after test is done

2015-03-27 Thread Ravi Prakash (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3324?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14384280#comment-14384280 ] Ravi Prakash commented on YARN-3324: And to clarify, I see the image as a dependency

[jira] [Commented] (YARN-3324) TestDockerContainerExecutor should clean test docker image from local repository after test is done

2015-03-27 Thread Ravindra Kumar Naik (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3324?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14385148#comment-14385148 ] Ravindra Kumar Naik commented on YARN-3324: --- Thanks [~raviprak] , [airbots] . I

[jira] [Commented] (YARN-3324) TestDockerContainerExecutor should clean test docker image from local repository after test is done

2015-03-26 Thread Chen He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3324?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14383323#comment-14383323 ] Chen He commented on YARN-3324: --- +1, sounds good to me. Thanks, [~ravindra.naik]

[jira] [Commented] (YARN-3324) TestDockerContainerExecutor should clean test docker image from local repository after test is done

2015-03-26 Thread Chen He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3324?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14383324#comment-14383324 ] Chen He commented on YARN-3324: --- Make sure there is no side effect if there are parallel

[jira] [Commented] (YARN-3324) TestDockerContainerExecutor should clean test docker image from local repository after test is done

2015-03-11 Thread Ravindra Naik (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3324?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14357300#comment-14357300 ] Ravindra Naik commented on YARN-3324: - Do you think that these steps will be sufficient

[jira] [Commented] (YARN-3324) TestDockerContainerExecutor should clean test docker image from local repository after test is done

2015-03-11 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3324?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14357313#comment-14357313 ] Hadoop QA commented on YARN-3324: - {color:green}+1 overall{color}. Here are the results of

[jira] [Commented] (YARN-3324) TestDockerContainerExecutor should clean test docker image from local repository after test is done

2015-03-11 Thread Chen He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3324?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14357249#comment-14357249 ] Chen He commented on YARN-3324: --- Hi [~ravindra.naik], thank you for the patch. Actually, just

[jira] [Commented] (YARN-3324) TestDockerContainerExecutor should clean test docker image from local repository after test is done

2015-03-11 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3324?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14356709#comment-14356709 ] Hadoop QA commented on YARN-3324: - {color:red}-1 overall{color}. Here are the results of