Hello Fang-Yu Rao, Qifan Chen, wangsheng, Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/19421

to look at the new patch set (#2).

Change subject: IMPALA-11829 - Fix bug in cardinality estimates related to 
TABLE_NUM_ROWS hint
......................................................................

IMPALA-11829 - Fix bug in cardinality estimates related to TABLE_NUM_ROWS hint

IMPALA-7942 added support for a TABLE_NUM_ROWS query hint which can be used
to specify a table cardinality for cases where stats are missing or invalid.
In the case where stats were missing or invalid and no TABLE_NUM_ROWS hint was 
specified
by the user, HdfsScanNode.getStatsNumRows was incorrectly returning a default 
value of -1
instead of returning a rough estimate of cardinality as it had prior to the 
IMPALA-7942 change.
This change fixes the return value of getStatsNumRows so it only uses the 
TABLE_NUM_ROWS
value when the users has actually specified the query hint.

Change-Id: Ia27745fd93abd5dec99bf82f16899bd15a2b88ae
---
M fe/src/main/java/org/apache/impala/planner/HdfsScanNode.java
1 file changed, 3 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/21/19421/2
--
To view, visit http://gerrit.cloudera.org:8080/19421
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ia27745fd93abd5dec99bf82f16899bd15a2b88ae
Gerrit-Change-Number: 19421
Gerrit-PatchSet: 2
Gerrit-Owner: David Rorke <dro...@cloudera.com>
Gerrit-Reviewer: Fang-Yu Rao <fangyu....@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Qifan Chen <qfc...@hotmail.com>
Gerrit-Reviewer: wangsheng <sky...@163.com>

Reply via email to