Quanlong Huang has posted comments on this change. ( http://gerrit.cloudera.org:8080/16428 )
Change subject: IMPALA-8291: Show constraints in DESCRIBE FORMATTED ...................................................................... Patch Set 1: (4 comments) The solution looks good to me. Just realized that most of the FE changes are copied from hive codes. I think we can merge it after checking the added constraints for tpcds. http://gerrit.cloudera.org:8080/#/c/16428/1/fe/src/compat-hive-3/java/org/apache/impala/compat/HiveMetadataFormatUtils.java File fe/src/compat-hive-3/java/org/apache/impala/compat/HiveMetadataFormatUtils.java: http://gerrit.cloudera.org:8080/#/c/16428/1/fe/src/compat-hive-3/java/org/apache/impala/compat/HiveMetadataFormatUtils.java@118 PS1, Line 118: if (!isFormatted) { : for (int i = 0; i < fields.length; i++) { : Object value = StringEscapeUtils.escapeJava(fields[i]); : if (value != null) { : tableInfo.append(value); : } : tableInfo.append((i == fields.length - 1) ? LINE_DELIM : FIELD_DELIM); : } Looks like these are dead codes since isFormatted is always true. Are there any purposes for adding them? Maybe we don't need them as commented at line 59. http://gerrit.cloudera.org:8080/#/c/16428/1/fe/src/compat-hive-3/java/org/apache/impala/compat/HiveMetadataFormatUtils.java@423 PS1, Line 423: .intern() nit: unneccessary intern http://gerrit.cloudera.org:8080/#/c/16428/1/fe/src/compat-hive-3/java/org/apache/impala/compat/HiveMetadataFormatUtils.java@475 PS1, Line 475: nit: blank line http://gerrit.cloudera.org:8080/#/c/16428/1/fe/src/compat-hive-3/java/org/apache/impala/compat/HiveMetadataFormatUtils.java@588 PS1, Line 588: nit: blank line -- To view, visit http://gerrit.cloudera.org:8080/16428 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I676b69c465c46491f870d7fdc894e7474c030356 Gerrit-Change-Number: 16428 Gerrit-PatchSet: 1 Gerrit-Owner: Shant Hovsepian <sh...@cloudera.com> Gerrit-Reviewer: David Rorke <dro...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com> Gerrit-Reviewer: Shant Hovsepian <sh...@cloudera.com> Gerrit-Reviewer: Vihang Karajgaonkar <vih...@cloudera.com> Gerrit-Comment-Date: Fri, 11 Sep 2020 08:19:48 +0000 Gerrit-HasComments: Yes
