Lars Volker has uploaded a new change for review. http://gerrit.cloudera.org:8080/4126
Change subject: IMPALA-3776: fix 'describe formatted' for Avro tables ...................................................................... IMPALA-3776: fix 'describe formatted' for Avro tables For Avro tables the column information in the underlying database of the Hive metastore can be different from what is specified in the avro schema. HIVE-6308 aimed to improve upon this, but for older tables the two don't necessarily align. There are two possible cases: 1) Hive's underlying database contains a column which is not present in the Avro schema file. In this case we encounter a NullPointerException in DescribeResultFactory.java#L189 when trying to look up the column in the internal table object. 2) The avro schema contains a column, which is not present in the underlying database. In this case the column will not be displayed in describe formatted. In addition to the automatic tests I verified this manually by creating an Avro table with an external schema file in Hive. This populated the underlying database with the column information. I then either removed a column from the Avro schema file (case 1) or cleared the column information from the "COLUMNS_V2" table in the underlying database (case 2) and verified that the change fixed both cases. Change-Id: Ieb69d3678e662465d40aee80ba23132ea13871a0 --- M fe/src/main/java/com/cloudera/impala/catalog/Column.java M fe/src/main/java/com/cloudera/impala/catalog/Table.java M fe/src/main/java/com/cloudera/impala/service/DescribeResultFactory.java M testdata/workloads/functional-query/queries/QueryTest/avro-schema-changes.test M tests/query_test/test_avro_schema_resolution.py 5 files changed, 100 insertions(+), 13 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/26/4126/1 -- To view, visit http://gerrit.cloudera.org:8080/4126 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ieb69d3678e662465d40aee80ba23132ea13871a0 Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Lars Volker <[email protected]>
