-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18831/
-----------------------------------------------------------
(Updated March 6, 2014, 8:24 a.m.)
Review request for Tajo.
Changes
-------
Hi Hyunsik.
Thank you for your review and detailed comments.
I had forgotten PreLogicalPlanVerifier.
As you mentioned, I updated the patch.
Please review it, again. :-)
Bugs: TAJO-663
https://issues.apache.org/jira/browse/TAJO-663
Repository: tajo
Description
-------
RawFile must be used for Intermediate file. So users can't use TABLE's File
Format. Bellow statement expected fail but runs successfully.
create table orders_100_raw (
o_orderkey INT8,
o_month text,
o_custkey INT8,
o_orderstatus TEXT,
o_totalprice FLOAT8,
o_orderdate TEXT,
o_orderpriority TEXT,
o_clerk TEXT,
o_shippriority INT4,
o_comment TEXT)
using RAW
Diffs (updated)
-----
tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/planner/PreLogicalPlanVerifier.java
024a9ae
tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/engine/eval/ExprTestBase.java
622da90
tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/engine/eval/TestPredicates.java
86b921f
tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/engine/function/TestStringOperatorsAndFunctions.java
f1d0ce3
Diff: https://reviews.apache.org/r/18831/diff/
Testing
-------
mvn clean install
Thanks,
Jung JaeHwa
