[jira] [Commented] (TEZ-1372) Fix preWarm to work after recent API changes

2014-08-08 Thread Bikas Saha (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14090982#comment-14090982 ] Bikas Saha commented on TEZ-1372: - Yes. That what I intend to do per comment above.

[jira] [Commented] (TEZ-1372) Fix preWarm to work after recent API changes

2014-08-08 Thread Gopal V (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14091011#comment-14091011 ] Gopal V commented on TEZ-1372: -- Missed PreWarmVertex.java in diff? Fix preWarm to work after

[jira] [Commented] (TEZ-1372) Fix preWarm to work after recent API changes

2014-08-07 Thread Bikas Saha (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14089513#comment-14089513 ] Bikas Saha commented on TEZ-1372: - If you want to override the sleep to something else then

[jira] [Commented] (TEZ-1372) Fix preWarm to work after recent API changes

2014-08-07 Thread Thaddeus Diamond (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14089584#comment-14089584 ] Thaddeus Diamond commented on TEZ-1372: --- Yes we are sure of that. Thanks, this works.

[jira] [Commented] (TEZ-1372) Fix preWarm to work after recent API changes

2014-08-07 Thread Gopal V (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14089848#comment-14089848 ] Gopal V commented on TEZ-1372: -- [~bikassaha]: getting to this after HIVE-7601 and HIVE-7639

[jira] [Commented] (TEZ-1372) Fix preWarm to work after recent API changes

2014-08-06 Thread Bikas Saha (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14088178#comment-14088178 ] Bikas Saha commented on TEZ-1372: - [~gopalv] [~hitesh] Review please! Fix preWarm to work

[jira] [Commented] (TEZ-1372) Fix preWarm to work after recent API changes

2014-08-06 Thread Thaddeus Diamond (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14088545#comment-14088545 ] Thaddeus Diamond commented on TEZ-1372: --- Woops! I was trying to make it more

[jira] [Commented] (TEZ-1372) Fix preWarm to work after recent API changes

2014-08-06 Thread Bikas Saha (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14088757#comment-14088757 ] Bikas Saha commented on TEZ-1372: - The intent is to remove the sleep. If its added as an API

[jira] [Commented] (TEZ-1372) Fix preWarm to work after recent API changes

2014-08-06 Thread Thaddeus Diamond (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14088761#comment-14088761 ] Thaddeus Diamond commented on TEZ-1372: --- Okay great. Set that config up and will

[jira] [Commented] (TEZ-1372) Fix preWarm to work after recent API changes

2014-08-05 Thread Thaddeus Diamond (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14086221#comment-14086221 ] Thaddeus Diamond commented on TEZ-1372: --- I like [~gopalv]'s idea, but am happy to make

[jira] [Commented] (TEZ-1372) Fix preWarm to work after recent API changes

2014-08-05 Thread Bikas Saha (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14086579#comment-14086579 ] Bikas Saha commented on TEZ-1372: - It will still be session.preWarm(FOO) where FOO is Vertex

[jira] [Commented] (TEZ-1372) Fix preWarm to work after recent API changes

2014-08-05 Thread Thaddeus Diamond (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14086839#comment-14086839 ] Thaddeus Diamond commented on TEZ-1372: --- This looks right on. I'll try it tomorrow

[jira] [Commented] (TEZ-1372) Fix preWarm to work after recent API changes

2014-08-05 Thread Bikas Saha (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14087139#comment-14087139 ] Bikas Saha commented on TEZ-1372: - That latest patch builds fine for me. Fix preWarm to

[jira] [Commented] (TEZ-1372) Fix preWarm to work after recent API changes

2014-08-05 Thread Thaddeus Diamond (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14087162#comment-14087162 ] Thaddeus Diamond commented on TEZ-1372: --- Sorry, grabbed the first one accidentally.

[jira] [Commented] (TEZ-1372) Fix preWarm to work after recent API changes

2014-08-05 Thread Bikas Saha (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14087174#comment-14087174 ] Bikas Saha commented on TEZ-1372: - TestContainerReuse is known to occasionally fail. The