[jira] [Commented] (HIVE-10005) remove some unnecessary branches from the inner loop

["Hive QA (JIRA)" <ji...@apache.org>]

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

Hive QA commented on HIVE-10005:
--------------------------------



{color:red}Overall{color}: -1 at least one tests failed

Here are the results of testing the latest attachment:
https://issues.apache.org/jira/secure/attachment/12723468/HIVE-10005.4.patch

{color:red}ERROR:{color} -1 due to 1 failed/errored test(s), 8701 tests executed
*Failed tests:*
{noformat}
TestMinimrCliDriver-smb_mapjoin_8.q - did not produce a TEST-*.xml file
{noformat}

Test results: http://ec2-174-129-184-35.compute-1.amazonaws.com/jenkins/job/PreCommit-HIVE-TRUNK-Build/3305/testReport
Console output: http://ec2-174-129-184-35.compute-1.amazonaws.com/jenkins/job/PreCommit-HIVE-TRUNK-Build/3305/console
Test logs: http://ec2-174-129-184-35.compute-1.amazonaws.com/logs/PreCommit-HIVE-TRUNK-Build-3305/

Messages:
{noformat}
Executing org.apache.hive.ptest.execution.PrepPhase
Executing org.apache.hive.ptest.execution.ExecutionPhase
Executing org.apache.hive.ptest.execution.ReportingPhase
Tests exited with: TestsFailedException: 1 tests failed
{noformat}

This message is automatically generated.

ATTACHMENT ID: 12723468 - PreCommit-HIVE-TRUNK-Build

> remove some unnecessary branches from the inner loop
> ----------------------------------------------------
>
>                 Key: HIVE-10005
>                 URL: https://issues.apache.org/jira/browse/HIVE-10005
>             Project: Hive
>          Issue Type: Bug
>            Reporter: Gunther Hagleitner
>            Assignee: Gunther Hagleitner
>         Attachments: HIVE-10005.1.patch, HIVE-10005.2.patch, HIVE-10005.3.patch, HIVE-10005.4.patch
>
>
> Operator.forward is doing too much. There's no reason to do the "done" checking per row and update it inline. It's much more efficient to just do that when the event that completes an operator happens.



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