impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Huaisi Xu (Code Review)" <ger...@cloudera.org>
Subject [Impala-CR](cdh5-2.6.0_5.8.0) IMPALA-3092: Set default value to NULL in AvroSchemaConverter
Date Fri, 27 May 2016 02:37:53 GMT
Hello Alex Behm,

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

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

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

Change subject: IMPALA-3092: Set default value to NULL in AvroSchemaConverter
......................................................................

IMPALA-3092: Set default value to NULL in AvroSchemaConverter

This change ensures that Avro tables created without column definitions
remain queryable if columns are added via ALTER TABLE. The bug was that
when synthesizing an Avro schema from the column definitions we used to
not add default values.

Change-Id: Ib86e9ba1f4329b285ae14ee299365f7291a7410e
---
M fe/src/main/java/com/cloudera/impala/util/AvroSchemaConverter.java
A testdata/workloads/functional-query/queries/QueryTest/avro-schema-changes.test
D testdata/workloads/functional-query/queries/QueryTest/avro-stale-schema.test
M tests/query_test/test_avro_schema_resolution.py
4 files changed, 79 insertions(+), 45 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/19/3219/9
-- 
To view, visit http://gerrit.cloudera.org:8080/3219
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ib86e9ba1f4329b285ae14ee299365f7291a7410e
Gerrit-PatchSet: 9
Gerrit-Project: Impala
Gerrit-Branch: cdh5-2.6.0_5.8.0
Gerrit-Owner: Huaisi Xu <hxu@cloudera.com>
Gerrit-Reviewer: Alex Behm <alex.behm@cloudera.com>
Gerrit-Reviewer: Bharath Vissapragada <bharathv@cloudera.com>
Gerrit-Reviewer: Huaisi Xu <hxu@cloudera.com>

Mime
View raw message