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

Thaddeus Diamond commented on TEZ-1372:
---------------------------------------

Very nice.  It's working.  Got it to run with:

{noformat}
TezClient tezClient = new TezClient(sessionName, tezConfiguration);
tezClient.start();

ProcessorDescriptor preWarmProcDescriptor = new 
ProcessorDescriptor(QueryPreWarmProcessor.class.getName());
Resource resource = Resource.newInstance(tezVertexMemoryMb, tezVertexVCores);
Vertex vertex = new Vertex(MyDummyProcessor.class.getName(), 
preWarmProcDescriptor, preWarmNumberOfContainers, resource);

tezClient.preWarm(vertex);
tezClient.waitTillReady();
{noformat}

Does that look correct to you guys procedurally?  Do I need the last line? m2c 
is that I don't need any more sugar than that as an app developer.  Makes sense 
semantically too, I use my application's processor and normal description to 
prewarm the client.  For good measure I could cache the {{ProcessorDescriptor}} 
and {{Resource}} in my example and reuse them later in the application but just 
change the I/O and {{Vertex}} object.

> 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