Paul Rogers has posted comments on this change. ( http://gerrit.cloudera.org:8080/12248 )
Change subject: IMPALA-8095: Detailed expression cardinality tests ...................................................................... Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/12248/7/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/7/fe/src/test/java/org/apache/impala/analysis/ExprCardinalityTest.java@122 PS7, Line 122: // Bug: NDV of partition columns is -1 though it is listed as : // 2 in the shell with: SHOW COLUMN STATS alltypes : //verifyTableCol(allTypes, "year", 2, 0); : // Bug: When tests are run in Eclipse we get the result above. : // But, when the same test is run using maven from the command line, : // we get the result shown below. : // Unit test in Eclipse see the above, unit tests run from the : // Disabling both to avoid a flaky test, : // Same issue for the next three tests. : //verifyTableCol(allTypes, "year", -1, -1); : //verifyTableCol(allTypes, "month", 12, 0); : //verifyTableCol(allTypes, "month", -1, -1); > (This and multiple places below) We should. We should also dig into the many bugs here. That said, the purpose of this patch is to create the test baseline, pointing out the many things to fix, including this one. OK to create the baseline first, then do the fixes as separate patches? -- 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: 7 Gerrit-Owner: Paul Rogers <prog...@cloudera.com> Gerrit-Reviewer: Bharath Vissapragada <bhara...@cloudera.com> Gerrit-Reviewer: Fredy Wijaya <fwij...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Paul Rogers <prog...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Comment-Date: Fri, 08 Feb 2019 22:30:38 +0000 Gerrit-HasComments: Yes
