Hello Bharath Vissapragada,

I'd like you to reexamine a change.  Please visit

    http://gerrit.cloudera.org:8080/3331

to look at the new patch set (#6).

Change subject: IMPALA-3687: Prefer Avro field name during schema reconciliation
......................................................................

IMPALA-3687: Prefer Avro field name during schema reconciliation

Currently Impala prefers schema definition from schema definition
except for CHAR/VARCHAR/STRING columns. As a result, Impala ignores
schema changes made in avro.schema.url or avro.schema.literal.

This patch prefers name, comment from Avro schema definition and
uses column type from column definition for CHAR/VARCHAR/STRING
columns.

Change-Id: Ia3e43b2885853c2b4f207a45a873c9d7f31379cd
---
M fe/src/main/java/com/cloudera/impala/util/AvroSchemaConverter.java
M fe/src/main/java/com/cloudera/impala/util/AvroSchemaUtils.java
M testdata/workloads/functional-query/queries/QueryTest/avro-schema-changes.test
M 
testdata/workloads/functional-query/queries/QueryTest/avro-schema-resolution.test
M tests/query_test/test_avro_schema_resolution.py
5 files changed, 74 insertions(+), 18 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/31/3331/6
-- 
To view, visit http://gerrit.cloudera.org:8080/3331
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ia3e43b2885853c2b4f207a45a873c9d7f31379cd
Gerrit-PatchSet: 6
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Huaisi Xu <[email protected]>
Gerrit-Reviewer: Alex Behm <[email protected]>
Gerrit-Reviewer: Bharath Vissapragada <[email protected]>
Gerrit-Reviewer: Dan Hecht <[email protected]>
Gerrit-Reviewer: Dimitris Tsirogiannis <[email protected]>
Gerrit-Reviewer: Huaisi Xu <[email protected]>

Reply via email to