----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17728/#review33677 -----------------------------------------------------------
tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/master/querymaster/Repartitioner.java <https://reviews.apache.org/r/17728/#comment63256> It will be set by scan node length. Its length is fixed by 2. Will it be a problem in future? How about update as follows: long[] stats = new long[scans.length]; - Jung JaeHwa On Feb. 5, 2014, 1:37 a.m., Hyunsik Choi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/17728/ > ----------------------------------------------------------- > > (Updated Feb. 5, 2014, 1:37 a.m.) > > > Review request for Tajo. > > > Bugs: TAJO-583 > https://issues.apache.org/jira/browse/TAJO-583 > > > Repository: tajo > > > Description > ------- > > Broadcast join does not work on partitioned tables. This is because broadcast > join condition just checks ScanNode whose data volume is smaller than some > threshold. We should also consider PartitionedTableScan. > > > Diffs > ----- > > > tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/planner/global/GlobalPlanner.java > cb2ac1536791626ae151dc5ac62e5d293c90813d > > tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/master/querymaster/Repartitioner.java > 0f4a62b319805b89f6aebc6e49b82aa3ee11d3a8 > > tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/master/querymaster/SubQuery.java > a2291690a3f77ffc08013d556a1d49ab3828ce60 > > tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/engine/query/TestTablePartitions.java > 46c1d94ef8c3824ccefedd00fe7e074e0537ecb6 > > tajo-core/tajo-core-backend/src/test/resources/queries/TestTablePartitions/case3.sql > PRE-CREATION > > tajo-core/tajo-core-backend/src/test/resources/results/TestTablePartitions/case3.result > PRE-CREATION > > Diff: https://reviews.apache.org/r/17728/diff/ > > > Testing > ------- > > mvn clean install > > > Thanks, > > Hyunsik Choi > >
