Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12248 )
Change subject: IMPALA-8095: Detailed expression cardinality tests ...................................................................... Patch Set 2: (6 comments) http://gerrit.cloudera.org:8080/#/c/12248/2/fe/src/test/java/org/apache/impala/analysis/ExprCardinalityTest.java File fe/src/test/java/org/apache/impala/analysis/ExprCardinalityTest.java: http://gerrit.cloudera.org:8080/#/c/12248/2/fe/src/test/java/org/apache/impala/analysis/ExprCardinalityTest.java@356 PS2, Line 356: private void verifyInequalitySel(String table, String col, String value) throws ImpalaException { line too long (99 > 90) http://gerrit.cloudera.org:8080/#/c/12248/2/fe/src/test/java/org/apache/impala/analysis/ExprCardinalityTest.java@462 PS2, Line 462: verifySelectExpr("alltypes", "int_col in (1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12)", 3, 1); line too long (93 > 90) http://gerrit.cloudera.org:8080/#/c/12248/2/fe/src/test/java/org/apache/impala/analysis/ExprCardinalityTest.java@490 PS2, Line 490: verifySelectExpr("alltypes", "int_col not in (1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12)", 3, 0); line too long (97 > 90) http://gerrit.cloudera.org:8080/#/c/12248/2/fe/src/test/java/org/apache/impala/common/FrontendFixture.java File fe/src/test/java/org/apache/impala/common/FrontendFixture.java: http://gerrit.cloudera.org:8080/#/c/12248/2/fe/src/test/java/org/apache/impala/common/FrontendFixture.java@340 PS2, Line 340: return ctx.analyzeAndAuthorize(parsedStmt, stmtTableCache, frontend_.getAuthzChecker()); line too long (92 > 90) http://gerrit.cloudera.org:8080/#/c/12248/2/fe/src/test/java/org/apache/impala/planner/CardinalityTest.java File fe/src/test/java/org/apache/impala/planner/CardinalityTest.java: http://gerrit.cloudera.org:8080/#/c/12248/2/fe/src/test/java/org/apache/impala/planner/CardinalityTest.java@106 PS2, Line 106: verifyCardinality("SELECT null_int FROM functional.nullrows WHERE group_str = 'x'", 4); line too long (91 > 90) http://gerrit.cloudera.org:8080/#/c/12248/2/fe/src/test/java/org/apache/impala/planner/CardinalityTest.java@110 PS2, Line 110: //verifyCardinality("SELECT null_int FROM functional.nullrows WHERE null_str = 'x'", 26); line too long (93 > 90) -- To view, visit http://gerrit.cloudera.org:8080/12248 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I3da58ee9b0beebeffb170b9430bd36d20dcd2401 Gerrit-Change-Number: 12248 Gerrit-PatchSet: 2 Gerrit-Owner: Paul Rogers <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Paul Rogers <[email protected]> Gerrit-Comment-Date: Tue, 22 Jan 2019 06:59:06 +0000 Gerrit-HasComments: Yes
