[ 
https://issues.apache.org/jira/browse/TEZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14087970#comment-14087970
 ] 

Bikas Saha commented on TEZ-1372:
---------------------------------

Thanks for trying it out. 

Why are you using MyDummyProcessor.class.getName() for the name? It does not 
matter to us, unless you are trying to use it as some key look up that may be 
better done via some API. Is QueryPreWarmProcessor deriving from the built-in 
PreWarmProcessor? It may be better in some cases where PreWarmProcessor may be 
able to initialize Tez lib classes. Also, PreWarmProcessor sleeps for some 
time. This hack is useful when we dont get containers quickly enough from YARN. 
Without the sleep the tasks might finish very quickly and container reuse may 
kick in. So we end up not acquiring preWarmNumberOfContainers.

> Fix preWarm to work after recent API changes
> --------------------------------------------
>
>                 Key: TEZ-1372
>                 URL: https://issues.apache.org/jira/browse/TEZ-1372
>             Project: Apache Tez
>          Issue Type: Sub-task
>    Affects Versions: 0.5.0
>            Reporter: Bikas Saha
>            Assignee: Bikas Saha
>            Priority: Blocker
>         Attachments: TEZ-1372.1.patch, TEZ-1372.2.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to