Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/19421 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/19421 Reviewed-by: Qifan Chen <[email protected]> Reviewed-by: wangsheng <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M fe/src/main/java/org/apache/impala/planner/HdfsScanNode.java 1 file changed, 3 insertions(+), 2 deletions(-) Approvals: Qifan Chen: Looks good to me, but someone else must approve wangsheng: Looks good to me, approved Impala Public Jenkins: Verified -- 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: merged Gerrit-Change-Id: Ia27745fd93abd5dec99bf82f16899bd15a2b88ae Gerrit-Change-Number: 19421 Gerrit-PatchSet: 3 Gerrit-Owner: David Rorke <[email protected]> Gerrit-Reviewer: Fang-Yu Rao <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Qifan Chen <[email protected]> Gerrit-Reviewer: wangsheng <[email protected]>
