You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@calcite.apache.org by jc...@apache.org on 2019/01/03 01:08:48 UTC

[calcite] 01/03: [CALCITE-2727] MV rewriting bails out incorrectly when a view does not contain any table reference

This is an automated email from the ASF dual-hosted git repository.

jcamacho pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/calcite.git

commit d6fb6abd9a17b81a5d8d592860ecd5c2f20ce9b1
Author: LantaoJin <ji...@gmail.com>
AuthorDate: Thu Dec 6 11:14:01 2018 +0800

    [CALCITE-2727] MV rewriting bails out incorrectly when a view does not contain any table reference
    
    Close apache/calcite#957
---
 .../org/apache/calcite/rel/rules/AbstractMaterializedViewRule.java    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/core/src/main/java/org/apache/calcite/rel/rules/AbstractMaterializedViewRule.java b/core/src/main/java/org/apache/calcite/rel/rules/AbstractMaterializedViewRule.java
index 7eedc93..94c456f 100644
--- a/core/src/main/java/org/apache/calcite/rel/rules/AbstractMaterializedViewRule.java
+++ b/core/src/main/java/org/apache/calcite/rel/rules/AbstractMaterializedViewRule.java
@@ -301,8 +301,8 @@ public abstract class AbstractMaterializedViewRule extends RelOptRule {
           // Extract view table references
           final Set<RelTableRef> viewTableRefs = mq.getTableReferences(viewNode);
           if (viewTableRefs == null) {
-            // Bail out
-            return;
+            // Skip it
+            continue;
           }
 
           // Extract view tables