-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11925/
-----------------------------------------------------------
(Updated Aug. 8, 2013, 12:24 a.m.)
Review request for hive, Ashutosh Chauhan and Jakob Homan.
Changes
-------
Adding support for binary in type info to schema generation.
Bugs: HIVE-3159
https://issues.apache.org/jira/browse/HIVE-3159
Repository: hive-git
Description
-------
Problem:
Hive doesn't support to create a Avro-based table using HQL create table
command. It currently requires to specify Avro schema literal or schema file
name.
For multiple cases, it is very inconvenient for user.
Some of the un-supported use cases:
1. Create table ... <Avro-SERDE etc.> as SELECT ... from <NON-AVRO FILE>
2. Create table ... <Avro-SERDE etc.> as SELECT from <AVRO TABLE>
3. Create table without specifying Avro schema.
Diffs (updated)
-----
ql/src/test/queries/clientpositive/avro_create_as_select.q PRE-CREATION
ql/src/test/queries/clientpositive/avro_create_as_select2.q PRE-CREATION
ql/src/test/queries/clientpositive/avro_no_schema_test.q PRE-CREATION
ql/src/test/queries/clientpositive/avro_without_schema.q PRE-CREATION
ql/src/test/results/clientpositive/avro_create_as_select.q.out PRE-CREATION
ql/src/test/results/clientpositive/avro_create_as_select2.q.out PRE-CREATION
ql/src/test/results/clientpositive/avro_no_schema_test.q.out PRE-CREATION
ql/src/test/results/clientpositive/avro_without_schema.q.out PRE-CREATION
serde/src/java/org/apache/hadoop/hive/serde2/avro/AvroSerdeUtils.java 13848b6
serde/src/java/org/apache/hadoop/hive/serde2/avro/TypeInfoToSchema.java
PRE-CREATION
serde/src/test/org/apache/hadoop/hive/serde2/avro/TestAvroSerdeUtils.java
010f614
serde/src/test/org/apache/hadoop/hive/serde2/avro/TestTypeInfoToSchema.java
PRE-CREATION
Diff: https://reviews.apache.org/r/11925/diff/
Testing
-------
Wrote a new java Test class for a new Java class. Added a new test case into
existing java test class. In addition, there are 4 .q file for testing multiple
use-cases.
Thanks,
Mohammad Islam
