-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32435/
-----------------------------------------------------------

Review request for hive, Brock Noland, Szehon Ho, and Xuefu Zhang.


Bugs: HIVE-10027
    https://issues.apache.org/jira/browse/HIVE-10027


Repository: hive-git


Description
-------

This patch supports to use the "doc" tag description in Avro schema fields or 
the column comment in Arvo table DDL as table column comments. Currently they 
are using the default "from deserializer" hardcoded in Hive MetaStoreUtil.


Diffs
-----

  ql/src/test/queries/clientpositive/avro_comments.q PRE-CREATION 
  ql/src/test/queries/clientpositive/avro_compression_enabled.q 
cb6f173ccfa76420f07c7d7e69749482537127aa 
  ql/src/test/queries/clientpositive/avro_evolved_schemas.q 
59cb21053fbb366962e39126f4f6623b8d53c8e0 
  ql/src/test/queries/clientpositive/avro_joins.q 
6d79bf1c1b4b403b6f33047dbe56ad656a68f990 
  ql/src/test/results/clientpositive/avro_add_column.q.out 
3aae56225ad671e44291d566b4a2400a9ee4ead9 
  ql/src/test/results/clientpositive/avro_add_column2.q.out 
0e26cd59f3969898f0cd2d47ae4d271fe39824f3 
  ql/src/test/results/clientpositive/avro_add_column3.q.out 
0418b29ca95e71dca0b46bc3bd26bb1564192d1e 
  ql/src/test/results/clientpositive/avro_change_schema.q.out 
5d52d215471527b00c5a277ddb77160249ea44a1 
  ql/src/test/results/clientpositive/avro_comments.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/avro_compression_enabled.q.out 
bcb967a11516b33eac045d160f88946606ce75f7 
  ql/src/test/results/clientpositive/avro_decimal.q.out 
88268ceb3a5dddc50d48b502dba17647f8d8e5fe 
  ql/src/test/results/clientpositive/avro_decimal_native.q.out 
c8ae0fb2404c8891c4e307d82d1c2d7f180eae51 
  ql/src/test/results/clientpositive/avro_evolved_schemas.q.out 
2f81b1ab9a6a266318945135726de7d23c809d8c 
  ql/src/test/results/clientpositive/avro_joins.q.out 
7e47fe0f25ad9b90971c26fc1fee95ceb32c0862 
  ql/src/test/results/clientpositive/avro_joins_native.q.out 
ca22a7279bc285600ffa40f76707fdd7e94dbb23 
  ql/src/test/results/clientpositive/avro_native.q.out 
2de4df1b8452e0b091a8ce3b6f5a9369ff208290 
  ql/src/test/results/clientpositive/avro_partitioned.q.out 
890cd9f764e35f9d99b42a7083f0bb49e2ec7bab 
  ql/src/test/results/clientpositive/avro_partitioned_native.q.out 
c02c656a45d54065280c4a30c0179f15d9a49267 
  ql/src/test/results/clientpositive/avro_sanity_test.q.out 
687579e69ba9d48bc471a89a7322f403eefce14a 
  ql/src/test/results/clientpositive/avro_schema_evolution_native.q.out 
84d8051422b032965dbd7f9b4e0892ead76f1b67 
  ql/src/test/results/clientpositive/avro_schema_literal.q.out 
b13694d16096a89c62aec82eceb057b47c2c4878 
  ql/src/test/results/clientpositive/create_like.q.out 
5e6836753effd84fc4e932ffc80a29b780613a23 
  ql/src/test/results/clientpositive/index_serde.q.out 
c0fc7e529a70ff8abfcc68bcd4e0e40e1cf87098 
  ql/src/test/results/clientpositive/spark/avro_decimal_native.q.out 
c8ae0fb2404c8891c4e307d82d1c2d7f180eae51 
  ql/src/test/results/clientpositive/spark/avro_joins.q.out 
7e47fe0f25ad9b90971c26fc1fee95ceb32c0862 
  ql/src/test/results/clientpositive/spark/avro_joins_native.q.out 
ca22a7279bc285600ffa40f76707fdd7e94dbb23 
  
serde/src/java/org/apache/hadoop/hive/serde2/avro/AvroObjectInspectorGenerator.java
 64a65433c7b476f8241e816e6c6fa20561872981 
  serde/src/java/org/apache/hadoop/hive/serde2/avro/AvroSerDe.java 
8ba0692c0a24b876f6087d90aceb744d1dcd94be 

Diff: https://reviews.apache.org/r/32435/diff/


Testing
-------

1. Manaul tests all passed
2. New qfile test avro_comments passed
3. Precommit build tests passed


Thanks,

Chaoyu Tang

Reply via email to