[
https://issues.apache.org/jira/browse/TEZ-3858?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16215707#comment-16215707
]
TezQA commented on TEZ-3858:
----------------------------
{color:red}-1 overall{color}. Here are the results of testing the latest
attachment
http://issues.apache.org/jira/secure/attachment/12893555/TEZ-3858.1.patch
against master revision bbe0f96.
{color:green}+1 @author{color}. The patch does not contain any @author
tags.
{color:red}-1 tests included{color}. The patch doesn't appear to include
any new or modified tests.
Please justify why no new tests are needed for this
patch.
Also please list what manual steps were performed to
verify this patch.
{color:green}+1 javac{color}. The applied patch does not increase the
total number of javac compiler warnings.
{color:green}+1 javadoc{color}. There were no new javadoc warning messages.
{color:green}+1 findbugs{color}. The patch does not introduce any new
Findbugs (version 3.0.1) warnings.
{color:green}+1 release audit{color}. The applied patch does not increase
the total number of release audit warnings.
{color:green}+1 core tests{color}. The patch passed unit tests in .
Test results:
https://builds.apache.org/job/PreCommit-TEZ-Build/2673//testReport/
Console output: https://builds.apache.org/job/PreCommit-TEZ-Build/2673//console
This message is automatically generated.
> Misleading dag level diagnostics in case of invalid vertex event
> ----------------------------------------------------------------
>
> Key: TEZ-3858
> URL: https://issues.apache.org/jira/browse/TEZ-3858
> Project: Apache Tez
> Issue Type: Bug
> Reporter: Zhiyuan Yang
> Assignee: Zhiyuan Yang
> Attachments: TEZ-3858.1.patch
>
>
> When a vertex gets invalid event, the state machine will be transited by
> InternalErrorTransition. This transition prints this and adds it to dag
> diagnostic:
> {code}
> ("Invalid event " + event.getType() + " on Vertex " +
> vertex.getLogIdentifier()
> {code}
> But variable event here is V_INTERNAL_ERROR event instead of the event that
> caused V_INTERNAL_ERROR. V_INTERNAL_ERROR is not the invalid event, the
> original event is.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
