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

Bikas Saha edited comment on TEZ-1568 at 9/23/14 5:51 PM:
----------------------------------------------------------

This is also not used. So perhaps just replace that with a Thread.sleep()?
{code}+        synchronized (condition) {
+          // wait for input event to initialize Reader
+          condition.wait();
+        }{code}
Or did you mean to use it like this comment suggests
{code}+        // otherwise it should call condition.notify()
+        throw new Exception(this.exLocation.name());{code}


was (Author: bikassaha):
This is also not used. So perhaps just replace that with a Thread.sleep()?
{code}+        synchronized (condition) {
+          // wait for input event to initialize Reader
+          condition.wait();
+        }{code}
Or did you mean to use it like this comment suggests
{code}

> Add system test for propagation of diagnostics for errors
> ---------------------------------------------------------
>
>                 Key: TEZ-1568
>                 URL: https://issues.apache.org/jira/browse/TEZ-1568
>             Project: Apache Tez
>          Issue Type: Sub-task
>            Reporter: Jeff Zhang
>            Assignee: Jeff Zhang
>         Attachments: TEZ-1568.patch, Tez-1568-2.patch, Tez-1568-3.patch, 
> Tez-1568-4.patch, Tez-1568-5.patch, Tez-1568-6.patch
>
>
> Design system test where exception come from Input, Output, Processor, 
> InputInitializer and VertexManagerPlugin



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to