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

Tajo QA commented on TAJO-609:
------------------------------

{color:red}*-1 overall.*{color}  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12629755/TAJO-609.patch
  against master revision c586227.

    {color:green}+1 @author.{color}  The patch does not contain any @author 
tags.

    {color:green}+1 tests included.{color}  The patch appears to include 2 new 
or modified test files.

    {color:green}+1 javac.{color}  The applied patch does not increase the 
total number of javac compiler warnings.

    {color:green}+1 javadoc.{color}  The applied patch does not increase the 
total number of javadoc warnings.

    {color:green}+1 checkstyle.{color}  The patch generated 0 code style errors.

    {color:red}-1 findbugs.{color}  The patch appears to introduce 173 new 
Findbugs (version 1.3.9) 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 
tajo-core/tajo-core-backend.

Test results: 
https://builds.apache.org/job/PreCommit-TAJO-Build/141//testReport/
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-TAJO-Build/141//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-core-backend.html
Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/141//console

This message is automatically generated.

> PlannerUtil::getRelationLineage ignores PartitionedTableScanNode
> ----------------------------------------------------------------
>
>                 Key: TAJO-609
>                 URL: https://issues.apache.org/jira/browse/TAJO-609
>             Project: Tajo
>          Issue Type: Bug
>          Components: planner/optimizer
>            Reporter: Hyunsik Choi
>            Assignee: Hyunsik Choi
>             Fix For: 0.8-incubating
>
>         Attachments: TAJO-609.patch
>
>
> Currently, PlannerUtil::getRelationLineage ignores PartitionedTableScanNode. 
> As a result, PlannerUtil::getRelationLineage can return empty relations. It 
> results in wrong estimation of relation size. Finally, in the case where a 
> broadcast join between a partitioned table and a small table, this problem 
> causes the bug to choose the larger table instead of smaller one to be loaded 
> on hash table.
> This patch fixes this bug, refactors inner-join related methods, and added 
> more comments on them.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Reply via email to