Huaisi Xu has posted comments on this change.

Change subject: IMPALA-3092: Avro table un-queriable after add columns without 
a default value
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/2361/1/tests/query_test/test_avro.py
File tests/query_test/test_avro.py:

Line 12: class TestAvroAddColumn(ImpalaTestSuite):
> Instead of creating a new test class, let's add to query_test/test_avro_sch
Sorry I did not brought up this yesterday.. alter table has not effect on avro 
tables using schema defined in url or literal. Actually Impala will throw 
exceptions when inserts columns with the same name twice. I suspect this to be 
impala inserting new column to metastore but later did not use meta store's 
definition. I think Impala should not write to meta store if the schema is in 
url or literal.. This is a separate issue.. i will create a Jira for this 
later. So I wont be able to include a record field, which as far as I know, 
exists in url or literal definition. Let me know if you have more suggestion!!


-- 
To view, visit http://gerrit.cloudera.org:8080/2361
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie86421fc3da51086e566998af317faa62ba9789b
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Huaisi Xu <[email protected]>
Gerrit-Reviewer: Huaisi Xu <[email protected]>
Gerrit-Reviewer: Skye Wanderman-Milne <[email protected]>
Gerrit-HasComments: Yes

Reply via email to