----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18556/#review35597 -----------------------------------------------------------
Ship it! +1 Thank you for your contribution. Ship it. - Hyunsik Choi On Feb. 27, 2014, 1:34 p.m., Jung JaeHwa wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/18556/ > ----------------------------------------------------------- > > (Updated Feb. 27, 2014, 1:34 p.m.) > > > Review request for Tajo. > > > Bugs: TAJO-620 > https://issues.apache.org/jira/browse/TAJO-620 > > > Repository: tajo > > > Description > ------- > > A join query can cause IndexOutOfBoundsException if one of tables is empty. > The reproduction condition is very straightforward. We should fix it. > > > Diffs > ----- > > > tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/benchmark/TPCH.java > 2e12b1d > > tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/master/querymaster/Repartitioner.java > 0d3f95e > > tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/engine/query/TestJoinQuery.java > 3a95724 > > tajo-core/tajo-core-backend/src/test/resources/queries/TestJoinQuery/testInnerJoinWithEmptyTable.sql > PRE-CREATION > > tajo-core/tajo-core-backend/src/test/resources/results/TestJoinQuery/testInnerJoinWithEmptyTable.result > PRE-CREATION > > Diff: https://reviews.apache.org/r/18556/diff/ > > > Testing > ------- > > mvn clean install > > > Thanks, > > Jung JaeHwa > >
