impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Huaisi Xu (Code Review)" <>
Subject [Impala-CR](cdh5-trunk) IMPALA-3687: Prefer Avro field name during schema reconciliation
Date Fri, 08 Jul 2016 20:43:10 GMT
Hello Bharath Vissapragada, Alex Behm,

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

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

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

IMPALA-3687: Prefer Avro field name during schema reconciliation

Since it is possible to create an Avro table with both column
definitions and an Avro schema, Impala attempts to reconcile
inconsistencies in the two schema definitions, generally preferring the
Avro schema. The only exception to this rule was with
CHAR/VARCHAR/STRING columns, where the column definition was preferred
in order to support tables with CHAR/VARCHAR columns although Avro only
supports STRING. This exception is confusing because the name for such a
column will be taken from the column definition (and not from the Avro

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

Change-Id: Ia3e43b2885853c2b4f207a45a873c9d7f31379cd
M fe/src/main/java/com/cloudera/impala/util/
M fe/src/main/java/com/cloudera/impala/util/
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/
5 files changed, 131 insertions(+), 17 deletions(-)

  git pull ssh:// refs/changes/31/3331/12
To view, visit
To unsubscribe, visit

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ia3e43b2885853c2b4f207a45a873c9d7f31379cd
Gerrit-PatchSet: 12
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Huaisi Xu <>
Gerrit-Reviewer: Alex Behm <>
Gerrit-Reviewer: Bharath Vissapragada <>
Gerrit-Reviewer: Dan Hecht <>
Gerrit-Reviewer: Dimitris Tsirogiannis <>
Gerrit-Reviewer: Huaisi Xu <>
Gerrit-Reviewer: Juan Yu <>

View raw message