You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@calcite.apache.org by "Julian Hyde (Jira)" <ji...@apache.org> on 2019/08/23 15:37:00 UTC
[jira] [Commented] (CALCITE-3287) Union's getRowCount in
RelMdRowCount has not consider 'union all'
[ https://issues.apache.org/jira/browse/CALCITE-3287?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16914356#comment-16914356 ]
Julian Hyde commented on CALCITE-3287:
--------------------------------------
Looks like a bug.
> Union's getRowCount in RelMdRowCount has not consider 'union all'
> -----------------------------------------------------------------
>
> Key: CALCITE-3287
> URL: https://issues.apache.org/jira/browse/CALCITE-3287
> Project: Calcite
> Issue Type: Bug
> Components: core
> Reporter: Hong Shen
> Priority: Minor
>
> when I read the RelMdRowCount.java, I found that union's getRowCount has not consider 'union all', while in Union.estimateRowCount has. I am not sure whether this is a bug, if it is, I can help fix it.
> {code:java}
> RelMdRowCount.java
> public Double getRowCount(Union rel, RelMetadataQuery mq) {
> double rowCount = 0.0;
> for (RelNode input : rel.getInputs()) {
> Double partialRowCount = mq.getRowCount(input);
> if (partialRowCount == null) {
> return null;
> }
> rowCount += partialRowCount;
> }
> return rowCount;
> }
> Union.java
> @Override public double estimateRowCount(RelMetadataQuery mq) {
> double dRows = RelMdUtil.getUnionAllRowCount(mq, this);
> if (!all) {
> dRows *= 0.5;
> }
> return dRows;
> }
> {code}
--
This message was sent by Atlassian Jira
(v8.3.2#803003)