DaeMyung Kang added a comment - 28/Nov/13 17:32 It passed unittests I changed only NULL value is involved with TajoDataType.Type.NULL to NULL_TYPE.

Jinho Kim added a comment - 29/Nov/13 02:12 +1 Thank you for your contribution! I reviewed the patch. The patch looks good, and passes all unit tests

Jinho Kim added a comment - 29/Nov/13 02:32 I've removed some unused imports.Then, I've committed the patch, fixing it as resolved.

Hudson added a comment - 29/Nov/13 09:35 SUCCESS: Integrated in Tajo-trunk-postcommit #571 (See https://builds.apache.org/job/Tajo-trunk-postcommit/571/ ) TAJO-290 : TajoDataType.Type.NULL should be NULL_TYPE. (DaeMyung Kang via jinho) (jinossy: https://git-wip-us.apache.org/repos/asf?p=incubator-tajo.git&a=commit&h=2e27a0232ac1bf48fe277a22a818d282866216c7 ) tajo-core/tajo-core-storage/src/test/java/org/apache/tajo/storage/v2/TestStorages.java tajo-core/tajo-core-storage/src/main/java/org/apache/tajo/storage/BinarySerializeDeserialize.java tajo-core/tajo-core-storage/src/main/java/org/apache/tajo/storage/RowFile.java tajo-core/tajo-core-storage/src/test/java/org/apache/tajo/storage/TestStorages.java tajo-common/src/main/java/org/apache/tajo/datum/DatumFactory.java tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/eval/IsNullEval.java tajo-core/tajo-core-storage/src/main/java/org/apache/tajo/storage/trevni/TrevniAppender.java tajo-common/src/main/java/org/apache/tajo/datum/Float4Datum.java tajo-core/tajo-core-storage/src/main/java/org/apache/tajo/storage/TextSerializeDeserialize.java tajo-core/tajo-core-storage/src/main/java/org/apache/tajo/storage/RawFile.java tajo-common/src/main/java/org/apache/tajo/datum/TextDatum.java tajo-common/src/main/java/org/apache/tajo/datum/CharDatum.java tajo-common/src/main/java/org/apache/tajo/datum/Int2Datum.java tajo-common/src/main/java/org/apache/tajo/datum/Float8Datum.java tajo-common/src/main/java/org/apache/tajo/datum/Int8Datum.java tajo-core/tajo-core-storage/src/test/java/org/apache/tajo/storage/TestLazyTuple.java tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/eval/FunctionEval.java tajo-common/src/main/proto/DataTypes.proto tajo-common/src/main/java/org/apache/tajo/datum/Int4Datum.java CHANGES.txt tajo-core/tajo-core-storage/src/main/java/org/apache/tajo/storage/trevni/TrevniScanner.java tajo-common/src/main/java/org/apache/tajo/datum/NullDatum.java