-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18264/
-----------------------------------------------------------

Review request for Tajo.


Bugs: TAJO-609
    https://issues.apache.org/jira/browse/TAJO-609


Repository: tajo


Description
-------

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.


Diffs
-----

  
tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/planner/PhysicalPlannerImpl.java
 5583efdaf2f809d2cc47af13962d0a7c284924cc 
  
tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/planner/PlannerUtil.java
 624518b534a6214e2a2908e21ea977254b23368c 
  
tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/LocalTajoTestingUtility.java
 ae59d11f8f43b67c03948aa5d0f9f5c63e971982 
  
tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/engine/planner/physical/TestHashJoinExec.java
 48cd2655a5d09aedbed92ff6c15fa1cc07246d08 

Diff: https://reviews.apache.org/r/18264/diff/


Testing
-------


Thanks,

Hyunsik Choi

Reply via email to