Alex Behm has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9065 )

Change subject: IMPALA-5440 Add planner tests with extreme statistics values
......................................................................


Patch Set 14:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/9065/14//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/9065/14//COMMIT_MSG@7
PS14, Line 7: IMPALA-5440 Add planner tests with extreme statistics values
IMPALA-5440: ...


http://gerrit.cloudera.org:8080/#/c/9065/14/fe/src/test/java/org/apache/impala/planner/PlannerTestBase.java
File fe/src/test/java/org/apache/impala/planner/PlannerTestBase.java:

http://gerrit.cloudera.org:8080/#/c/9065/14/fe/src/test/java/org/apache/impala/planner/PlannerTestBase.java@670
PS14, Line 670:    * This function plans query and fails if estimated 
cardinalities are
This function plans the given query and fails if the estimated cardinalities 
are not within the specified bounds ...


http://gerrit.cloudera.org:8080/#/c/9065/14/fe/src/test/java/org/apache/impala/planner/PlannerTestBase.java@695
PS14, Line 695:           if (cardinality < min || cardinality > max){
* should not return an error for -1
* style: space before "{"



--
To view, visit http://gerrit.cloudera.org:8080/9065
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I86dec47cf1438882cafaec53e97864ccfcdff3cb
Gerrit-Change-Number: 9065
Gerrit-PatchSet: 14
Gerrit-Owner: Xinran Tinney <xyutin...@cloudera.com>
Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com>
Gerrit-Reviewer: Jim Apple <jbapple-imp...@apache.org>
Gerrit-Reviewer: Lars Volker <l...@cloudera.com>
Gerrit-Reviewer: Vuk Ercegovac <vercego...@cloudera.com>
Gerrit-Reviewer: Xinran Tinney <xyutin...@cloudera.com>
Gerrit-Comment-Date: Mon, 12 Feb 2018 23:23:14 +0000
Gerrit-HasComments: Yes

Reply via email to