-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20096/
-----------------------------------------------------------
(Updated April 14, 2014, 6:49 p.m.)
Review request for hive.
Changes
-------
Addressed Carl's comments. Changes:
- Reverted whitespace changes.
- Moved the TABLE_PROP_PREFIX ("table.") to serdeConstants.
- Removed code that mutated the Properties passed to the AvroSerDe
- Added/improved comments
- Synced with latest
Repository: hive-git
Description
-------
The problem occurs when you store the "avro.schema.(literal|url)" in the
SERDEPROPERTIES instead of the TBLPROPERTIES, add a partition, change the
table's schema, and then try reading from the old partition.
I fixed this problem by passing the table properties to the partition with a
"table." prefix, and changing the Avro SerDe to always use the table properties
when available.
Diffs (updated)
-----
ql/src/java/org/apache/hadoop/hive/ql/plan/PartitionDesc.java 43cef5c
ql/src/test/queries/clientpositive/avro_partitioned.q 6fe5117
ql/src/test/results/clientpositive/avro_partitioned.q.out 644716d
serde/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/serde/serdeConstants.java
515cf25
serde/src/java/org/apache/hadoop/hive/serde2/avro/AvroSerdeUtils.java 9d58d13
serde/src/test/org/apache/hadoop/hive/serde2/avro/TestAvroSerdeUtils.java
67d5570
Diff: https://reviews.apache.org/r/20096/diff/
Testing
-------
Added test cases
Thanks,
Anthony Hsu
