[
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 6:01 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}
Rest looks fine. If possible, please step through one of the complex cases in
the debugger to double check that the flow is as we expect it to be. Sometimes
tests work in unexpected ways :)
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}+ // otherwise it should call condition.notify()
+ throw new Exception(this.exLocation.name());{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)
