> On June 2, 2014, 2:26 p.m., Jihoon Son wrote: > > tajo-core/src/test/resources/results/TestTablePartitions/case14.result, > > line 2 > > <https://reviews.apache.org/r/21906/diff/2/?file=594932#file594932line2> > > > > In my opinion, it would be better to test queries which output some > > meaningful results.
Thanks Jihoo. To implement your opinion, I need to commit TAJO-840 patch for a start. If you will be able to review TAJO-840, it will be very helpful to me. Cheers Jaehwa > On June 2, 2014, 2:26 p.m., Jihoon Son wrote: > > tajo-core/src/test/resources/results/TestTablePartitions/case15.result, > > line 2 > > <https://reviews.apache.org/r/21906/diff/2/?file=594933#file594933line2> > > > > In my opinion, it would be better to test queries which output some > > meaningful results. See above comments. - Jung ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21906/#review44496 ----------------------------------------------------------- On May 27, 2014, 2:03 p.m., Jung JaeHwa wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/21906/ > ----------------------------------------------------------- > > (Updated May 27, 2014, 2:03 p.m.) > > > Review request for Tajo. > > > Bugs: TAJO-848 > https://issues.apache.org/jira/browse/TAJO-848 > > > Repository: tajo > > > Description > ------- > > Currently, user can insert data even if there is small columns compare to > target columns as follows: > create table table1 (id int, name text, score float, type text) using csv > with ('csvfile.delimiter'='|') ; > > create table table3 (id int, name text, score float, type text) using csv > with ('csvfile.delimiter'='|') ; > > create table table4 (id int, name text, score float, type text) using csv > with ('csvfile.delimiter'='|') partition by column(part text); > > default> insert overwrite into table3 select id, name from table1; > Progress: 0%, response time: 0.544 sec > Progress: 100%, response time: 1.158 sec > (5 rows, 1.158 sec, 60 B inserted) > > default> insert overwrite into table4 select id, name from table1; > Progress: 100%, response time: 0.468 sec > (5 rows, 0.468 sec, 65 B inserted) > As above, tajo can run a query for inserting data. But in partitioned table, > tajo made null value for partition column as follows: > hdfs dfs -ls /tajo/warehouse/default/table4 > Found 1 items > drwxr-xr-x - blrunner supergroup 0 2014-05-26 16:27 > /tajo/warehouse/default/table4/part= > Thus we need to make a syntax exception for blocking null partition value. > > > Diffs > ----- > > > tajo-core/src/main/java/org/apache/tajo/engine/planner/PreLogicalPlanVerifier.java > 5eca5fd > > tajo-core/src/test/java/org/apache/tajo/engine/query/TestTablePartitions.java > 0ec7de0 > tajo-core/src/test/resources/queries/TestTablePartitions/case14.sql > PRE-CREATION > tajo-core/src/test/resources/queries/TestTablePartitions/case15.sql > PRE-CREATION > tajo-core/src/test/resources/results/TestTablePartitions/case14.result > PRE-CREATION > tajo-core/src/test/resources/results/TestTablePartitions/case15.result > PRE-CREATION > > Diff: https://reviews.apache.org/r/21906/diff/ > > > Testing > ------- > > mvn clean install > > > Thanks, > > Jung JaeHwa > >
