Alex Behm has posted comments on this change. Change subject: IMPALA-3776: fix 'describe formatted' for Avro tables ......................................................................
Patch Set 1: Code-Review+2 (3 comments) http://gerrit.cloudera.org:8080/#/c/4126/1/testdata/workloads/functional-query/queries/QueryTest/avro-schema-changes.test File testdata/workloads/functional-query/queries/QueryTest/avro-schema-changes.test: Line 71: # IMPALA-3776: Create an avro table, add a column to the avro schema and make sure nit: Avro table (Avro is a proper noun) fix everywhere Line 77: "namespace": "com.cloudera.test", org.apache.test :) and below as well Line 133: # Describe formatted must not throw a NPE. remove comment, pretty clear that this should not NPE if we have a positive test for it -- To view, visit http://gerrit.cloudera.org:8080/4126 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ieb69d3678e662465d40aee80ba23132ea13871a0 Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Lars Volker <[email protected]> Gerrit-Reviewer: Alex Behm <[email protected]> Gerrit-Reviewer: Bharath Vissapragada <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-Reviewer: Lars Volker <[email protected]> Gerrit-Reviewer: Marcel Kornacker <[email protected]> Gerrit-HasComments: Yes
