Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/14750 )
Change subject: IMPALA-9092: Add support for creating external Kudu table ...................................................................... Patch Set 3: (33 comments) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/analysis/AnalyzeKuduDDLTest.java File fe/src/test/java/org/apache/impala/analysis/AnalyzeKuduDDLTest.java: http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/analysis/AnalyzeKuduDDLTest.java@71 PS3, Line 71: super.AnalysisError(appendSynchronizedTblProps(stmt, isSynchronizedTbl), expectedError); line too long (92 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/analysis/AnalyzeKuduDDLTest.java@102 PS3, Line 102: "range(x, y) (partition value = (1+1, 2+2), partition value = ((1+1+1)+1, 10), " + line too long (94 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/analysis/AnalyzeKuduDDLTest.java@221 PS3, Line 221: "(PARTITION VALUE = 'abc')' is not a key column. Only key columns can be used " line too long (91 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/analysis/AnalyzeKuduDDLTest.java@277 PS3, Line 277: "partition by range(a, b) (partition (0, 0) < values <= (1, 1)) stored as kudu", line too long (92 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/analysis/AnalyzeKuduDDLTest.java@286 PS3, Line 286: "partitioning columns: (1 vs 2). Range partition: 'PARTITION 0 < VALUES <= 1'", line too long (91 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java File fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java: http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java@266 PS3, Line 266: " id INT NOT NULL ENCODING AUTO_ENCODING COMPRESSION DEFAULT_COMPRESSION,\n" + line too long (93 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java@267 PS3, Line 267: " bool_col BOOLEAN NULL ENCODING AUTO_ENCODING COMPRESSION DEFAULT_COMPRESSION,\n" + line too long (99 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java@268 PS3, Line 268: " tinyint_col TINYINT NULL ENCODING AUTO_ENCODING COMPRESSION DEFAULT_COMPRESSION,\n" + line too long (102 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java@269 PS3, Line 269: " smallint_col SMALLINT NULL ENCODING AUTO_ENCODING COMPRESSION DEFAULT_COMPRESSION,\n" + line too long (104 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java@270 PS3, Line 270: " int_col INT NULL ENCODING AUTO_ENCODING COMPRESSION DEFAULT_COMPRESSION,\n" + line too long (94 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java@271 PS3, Line 271: " bigint_col BIGINT NULL ENCODING AUTO_ENCODING COMPRESSION DEFAULT_COMPRESSION,\n" + line too long (100 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java@272 PS3, Line 272: " float_col FLOAT NULL ENCODING AUTO_ENCODING COMPRESSION DEFAULT_COMPRESSION,\n" + line too long (98 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java@273 PS3, Line 273: " double_col DOUBLE NULL ENCODING AUTO_ENCODING COMPRESSION DEFAULT_COMPRESSION,\n" + line too long (100 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java@274 PS3, Line 274: " date_string_col STRING NULL ENCODING AUTO_ENCODING COMPRESSION DEFAULT_COMPRESSION,\n" + line too long (105 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java@275 PS3, Line 275: " string_col STRING NULL ENCODING AUTO_ENCODING COMPRESSION DEFAULT_COMPRESSION,\n" + line too long (100 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java@276 PS3, Line 276: " timestamp_col TIMESTAMP NULL ENCODING AUTO_ENCODING COMPRESSION DEFAULT_COMPRESSION,\n" + line too long (106 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java@277 PS3, Line 277: " year INT NULL ENCODING AUTO_ENCODING COMPRESSION DEFAULT_COMPRESSION,\n" + line too long (91 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java@278 PS3, Line 278: " month INT NULL ENCODING AUTO_ENCODING COMPRESSION DEFAULT_COMPRESSION,\n" + line too long (92 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java@288 PS3, Line 288: " id INT NOT NULL ENCODING AUTO_ENCODING COMPRESSION DEFAULT_COMPRESSION,\n" + line too long (93 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java@289 PS3, Line 289: " bool_col BOOLEAN NULL ENCODING AUTO_ENCODING COMPRESSION DEFAULT_COMPRESSION,\n" + line too long (99 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java@290 PS3, Line 290: " tinyint_col TINYINT NULL ENCODING AUTO_ENCODING COMPRESSION DEFAULT_COMPRESSION,\n" + line too long (102 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java@291 PS3, Line 291: " smallint_col SMALLINT NULL ENCODING AUTO_ENCODING COMPRESSION DEFAULT_COMPRESSION,\n" + line too long (104 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java@292 PS3, Line 292: " int_col INT NULL ENCODING AUTO_ENCODING COMPRESSION DEFAULT_COMPRESSION,\n" + line too long (94 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java@293 PS3, Line 293: " bigint_col BIGINT NULL ENCODING AUTO_ENCODING COMPRESSION DEFAULT_COMPRESSION,\n" + line too long (100 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java@294 PS3, Line 294: " float_col FLOAT NULL ENCODING AUTO_ENCODING COMPRESSION DEFAULT_COMPRESSION,\n" + line too long (98 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java@295 PS3, Line 295: " double_col DOUBLE NULL ENCODING AUTO_ENCODING COMPRESSION DEFAULT_COMPRESSION,\n" + line too long (100 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java@296 PS3, Line 296: " date_string_col STRING NULL ENCODING AUTO_ENCODING COMPRESSION DEFAULT_COMPRESSION,\n" + line too long (105 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java@297 PS3, Line 297: " string_col STRING NULL ENCODING AUTO_ENCODING COMPRESSION DEFAULT_COMPRESSION,\n" + line too long (100 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java@298 PS3, Line 298: " timestamp_col TIMESTAMP NULL ENCODING AUTO_ENCODING COMPRESSION DEFAULT_COMPRESSION,\n" + line too long (106 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java@299 PS3, Line 299: " year INT NULL ENCODING AUTO_ENCODING COMPRESSION DEFAULT_COMPRESSION,\n" + line too long (91 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java@300 PS3, Line 300: " month INT NULL ENCODING AUTO_ENCODING COMPRESSION DEFAULT_COMPRESSION,\n" + line too long (92 > 90) http://gerrit.cloudera.org:8080/#/c/14750/3/tests/query_test/test_kudu.py File tests/query_test/test_kudu.py: http://gerrit.cloudera.org:8080/#/c/14750/3/tests/query_test/test_kudu.py@1053 PS3, Line 1053: ' flake8: E501 line too long (95 > 90 characters) http://gerrit.cloudera.org:8080/#/c/14750/3/tests/query_test/test_kudu.py@1072 PS3, Line 1072: ' flake8: E501 line too long (95 > 90 characters) -- To view, visit http://gerrit.cloudera.org:8080/14750 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I76f81d41db0cf2269ee1b365857164a43677e14d Gerrit-Change-Number: 14750 Gerrit-PatchSet: 3 Gerrit-Owner: Vihang Karajgaonkar <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Vihang Karajgaonkar <[email protected]> Gerrit-Comment-Date: Thu, 21 Nov 2019 01:36:07 +0000 Gerrit-HasComments: Yes
