Raghav Jindal has uploaded this change for review. ( http://gerrit.cloudera.org:8080/23801
Change subject: IMPALA-11926: Only loading Avro schema in LocalFsTable if the table format is Avro ...................................................................... IMPALA-11926: Only loading Avro schema in LocalFsTable if the table format is Avro In the local catalog mode LocalFsTable was always trying to load the Avro schema if the avro.schema.url table property was present, even when the table format was not Avro. This made tables unloadable if the URL was invalid even when the table had no Avro data. This is a regression from legacy catalog mode where HdfsTable.setAvroSchema() only loads the Avro schema if the table format is Avro or if there are any Avro partitions. This change moves the loadAvroSchema() call inside the isAvroFormat() if condition check, matching the legacy behavior. Avro schemas for Avro partitions in mixed-format tables are still handled correctly in the toTHdfsTable(). Change-Id: I202665f978401894a1c837293529c06fa4270985 --- M fe/src/main/java/org/apache/impala/catalog/local/LocalFsTable.java 1 file changed, 12 insertions(+), 8 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/01/23801/1 -- To view, visit http://gerrit.cloudera.org:8080/23801 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I202665f978401894a1c837293529c06fa4270985 Gerrit-Change-Number: 23801 Gerrit-PatchSet: 1 Gerrit-Owner: Raghav Jindal <[email protected]>
