Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-4282: Remove max length check for type strings. ......................................................................
IMPALA-4282: Remove max length check for type strings. During analysis, Impala used to enforce the default HMS limit on the number of characters of a type string. That enforcement has the benefit of a clear error message, but has the severe drawback of not having any workaround because the limit is baked into the code. Going above the 4000 limit is pretty easy with complex types, and several users have already run into this issue. This patch removes the Impala-side enforcement such that the 4000 limit can be increased by altering the corresponding columns in the Metastore's backend database. Change-Id: I01f5e503e14feee857bbdf80b3ef4f5a8b57fb2d Reviewed-on: http://gerrit.cloudera.org:8080/6034 Reviewed-by: Alex Behm <[email protected]> Tested-by: Impala Public Jenkins --- M fe/src/main/java/org/apache/impala/analysis/ColumnDef.java M fe/src/main/java/org/apache/impala/util/MetaStoreUtil.java M fe/src/test/java/org/apache/impala/analysis/AnalyzeDDLTest.java 3 files changed, 1 insertion(+), 50 deletions(-) Approvals: Impala Public Jenkins: Verified Alex Behm: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/6034 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I01f5e503e14feee857bbdf80b3ef4f5a8b57fb2d Gerrit-PatchSet: 4 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Alex Behm <[email protected]> Gerrit-Reviewer: Alex Behm <[email protected]> Gerrit-Reviewer: Bharath Vissapragada <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-Reviewer: Impala Public Jenkins
