Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13311 )
Change subject: [IMPALA-8435] Prohibit operations on transactional table. ...................................................................... Patch Set 12: (2 comments) http://gerrit.cloudera.org:8080/#/c/13311/12/fe/src/main/java/org/apache/impala/analysis/Analyzer.java File fe/src/main/java/org/apache/impala/analysis/Analyzer.java: http://gerrit.cloudera.org:8080/#/c/13311/12/fe/src/main/java/org/apache/impala/analysis/Analyzer.java@207 PS12, Line 207: throw new AnalysisException(String.format(INSERT_ONLY_ACID_TABLE_SUPPORTED_ERROR_MSG, line too long (91 > 90) http://gerrit.cloudera.org:8080/#/c/13311/12/fe/src/main/java/org/apache/impala/util/AcidUtils.java File fe/src/main/java/org/apache/impala/util/AcidUtils.java: http://gerrit.cloudera.org:8080/#/c/13311/12/fe/src/main/java/org/apache/impala/util/AcidUtils.java@39 PS12, Line 39: // Ideally we should move the AcidUtils code from hive-exec into hive-standalone-metastore line too long (92 > 90) -- To view, visit http://gerrit.cloudera.org:8080/13311 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I740dc4ce0dbbc0c2e042b01832e606cc1ac4132a Gerrit-Change-Number: 13311 Gerrit-PatchSet: 12 Gerrit-Owner: Sudhanshu Arora <sudhan...@cloudera.com> Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Sudhanshu Arora <sudhan...@cloudera.com> Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Vihang Karajgaonkar <vih...@cloudera.com> Gerrit-Reviewer: Yongzhi Chen <yc...@cloudera.com> Gerrit-Comment-Date: Wed, 22 May 2019 05:24:20 +0000 Gerrit-HasComments: Yes
