> On Feb. 7, 2014, 4:03 p.m., Jung JaeHwa wrote: > > +1. > > It looks good overall. :)
Thanks for the quick review. - Hyunsik ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17728/#review33916 ----------------------------------------------------------- On Feb. 7, 2014, 2:12 p.m., Hyunsik Choi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/17728/ > ----------------------------------------------------------- > > (Updated Feb. 7, 2014, 2:12 p.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 > ----- > > CHANGES.txt 36c106211f3f1c224f58174e8e261d0102a70f38 > tajo-common/src/main/java/org/apache/tajo/util/TUtil.java > e9d82197671dabf6388e26a42a3a1bf37145e61c > > tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/planner/PhysicalPlannerImpl.java > b8c52c0147bae6e1f29cf3c71d5969709672e410 > > tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/planner/PlanString.java > ef8bed0d95c0ee92e7c37d9d9307a97f575c1733 > > tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/planner/PlannerUtil.java > bd9001c679f7d9b0914760c83ca10891c0ce0fd4 > > tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/planner/global/ExecutionBlock.java > 4f3976e804a621e8eee4aaca96097256f9cfc6b1 > > 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/engine/planner/global/MasterPlan.java > f6ac2befc2bc4c45e07cfafe6d01bf4c6f303412 > > tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/planner/physical/HashLeftOuterJoinExec.java > d0c989758334cfb3f2bd7605c118ec1271b8c31f > > tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/planner/physical/PartitionMergeScanExec.java > PRE-CREATION > > tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/planner/rewrite/FilterPushDownRule.java > 68a0e305673c0894977d70ac1ed838228a10ed59 > > tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/planner/rewrite/ProjectionPushDownRule.java > 7512f23533cce31a4f87e5badf0296da71806124 > > 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 > 02c2f6a5e264ab985be31bc7ef612ca2eb3f693e > > tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/LocalTajoTestingUtility.java > fa67eb694b082439fdd27f630329875a878f5c32 > > tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/QueryTestCaseBase.java > 24a4237ed3c46c87ee480e36a40402102dfe5531 > > tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/TajoTestingCluster.java > 6e73df88d6ab37c8466667a9a87a5f447ff3c03b > tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/TpchTestBase.java > ad3d67668b9c112f05f0af296a47f3e27ba2546b > > tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/engine/planner/physical/TestSortExec.java > 07f726db109d8aa45510adeb2cda25e9848b063e > > tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/engine/query/TestCTASQuery.java > ede0d584cfa9350d820e0ab64e3d1cf65b95b170 > > tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/engine/query/TestJoinOnPartitionedTables.java > PRE-CREATION > > 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/TestGroupByQuery/testGroupByNested1.sql > 10fa4239a79cb550873f7601fc176ceb5c7bdd7f > > tajo-core/tajo-core-backend/src/test/resources/queries/TestGroupByQuery/testGroupByNested2.sql > e7bbcac1dbd2207693adf09aa29ca2c06976e88e > > tajo-core/tajo-core-backend/src/test/resources/queries/TestJoinOnPartitionedTables/customer_ddl.sql > PRE-CREATION > > tajo-core/tajo-core-backend/src/test/resources/queries/TestJoinOnPartitionedTables/insert_into_customer.sql > PRE-CREATION > > tajo-core/tajo-core-backend/src/test/resources/queries/TestJoinOnPartitionedTables/selfJoinOfPartitionedTable.sql > PRE-CREATION > > tajo-core/tajo-core-backend/src/test/resources/queries/TestJoinOnPartitionedTables/testNoProjectionJoinQual.sql > PRE-CREATION > > tajo-core/tajo-core-backend/src/test/resources/queries/TestJoinOnPartitionedTables/testPartialFilterPushDown.sql > PRE-CREATION > > tajo-core/tajo-core-backend/src/test/resources/queries/TestJoinOnPartitionedTables/testPartitionTableJoinSmallTable.sql > PRE-CREATION > > tajo-core/tajo-core-backend/src/test/resources/queries/TestJoinQuery/testFullOuterJoin1.sql > 221b8a95c31fcb57c9a4c5bee1279714af7b6878 > > tajo-core/tajo-core-backend/src/test/resources/queries/TestJoinQuery/testLeftOuterJoin1.sql > 59a876af8e8d6dbbcacdf315f806cab9e48b7254 > > tajo-core/tajo-core-backend/src/test/resources/queries/TestJoinQuery/testLeftOuterJoinWithConstantExpr1.sql > 334c161f259f161967d735da12b032895ece560c > > tajo-core/tajo-core-backend/src/test/resources/queries/TestJoinQuery/testLeftOuterJoinWithConstantExpr2.sql > 3256e28160c3356d79ae414d89a3578b6ca986d0 > > tajo-core/tajo-core-backend/src/test/resources/queries/TestJoinQuery/testLeftOuterJoinWithConstantExpr3.sql > 03cdae2cdadec2042ae455c6c20f41437b0ee11b > > tajo-core/tajo-core-backend/src/test/resources/queries/TestJoinQuery/testRightOuterJoin1.sql > cf9896d6c253a8213de6761d9a1360d9b6ed8606 > > tajo-core/tajo-core-backend/src/test/resources/queries/TestTablePartitions/case3.sql > PRE-CREATION > > tajo-core/tajo-core-backend/src/test/resources/queries/default/create_partitioned_table_as_select.sql > 09b14eb0b6359119d6a48f4e3de1ab6428571cdd > > tajo-core/tajo-core-backend/src/test/resources/results/TestGroupByQuery/testGroupByNested1.result > 85c55983fd5b17adf9e432cfc2951ebffa524aa8 > > tajo-core/tajo-core-backend/src/test/resources/results/TestGroupByQuery/testGroupByNested2.result > edae882317118e0354b34543bdc98322a753a467 > > tajo-core/tajo-core-backend/src/test/resources/results/TestJoinOnPartitionedTables/selfJoinOfPartitionedTable.result > PRE-CREATION > > tajo-core/tajo-core-backend/src/test/resources/results/TestJoinOnPartitionedTables/testNoProjectionJoinQual.result > PRE-CREATION > > tajo-core/tajo-core-backend/src/test/resources/results/TestJoinOnPartitionedTables/testPartialFilterPushDown.result > PRE-CREATION > > tajo-core/tajo-core-backend/src/test/resources/results/TestJoinOnPartitionedTables/testPartitionTableJoinSmallTable.result > PRE-CREATION > > tajo-core/tajo-core-backend/src/test/resources/results/TestTablePartitions/case3.result > PRE-CREATION > tajo-storage/src/main/java/org/apache/tajo/storage/CSVFile.java > 5d05d6f1fe039f9f53b777ea3b1a61a136af133b > > Diff: https://reviews.apache.org/r/17728/diff/ > > > Testing > ------- > > mvn clean install > > > Thanks, > > Hyunsik Choi > >
