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

Jonathan Eagles commented on TEZ-1072:
--------------------------------------

[~sseth], [~bikassaha], Incorporating the feedback, much nicer with only 2 wait 
functions.

- removed waitForCompletionWithStatusUpdates
- renamed waitForCompletionWithAllStatusUpdates to 
waitForCompletionWithAllStatusUpdates
- made sure waitForCompletion does not log updates

> Consolidate monitoring APIs in DAGClient
> ----------------------------------------
>
>                 Key: TEZ-1072
>                 URL: https://issues.apache.org/jira/browse/TEZ-1072
>             Project: Apache Tez
>          Issue Type: Improvement
>            Reporter: Siddharth Seth
>            Assignee: Jonathan Eagles
>            Priority: Blocker
>              Labels: api
>         Attachments: TEZ-1072-DAGCLIENTUTILS-v1.patch, TEZ-1072-v1.patch
>
>
> Rename waitForCompletionWithAllStatusUpdates - was this meant to be 
> waitForCompletionWithAllVertexUpdates
> Reduce the number of methods exposed - waitForCompletion, 
> waitForCompletionWithStatusUpdates(@Nullable Set<Vertex> vertices,
>       @Nullable Set<StatusGetOpts> statusGetOpts), 
> waitForCompletionWithAllStatusUpdates(@Nullable Set<StatusGetOpts> 
> statusGetOpts)



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

Reply via email to