>From <[email protected]>:

[email protected] has uploaded this change for review. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17810 )


Change subject: [ASTERIXDB-3261][COMP] Limit Costing
......................................................................

[ASTERIXDB-3261][COMP] Limit Costing

Change-Id: Ifd03a57f893e7c22ee3bc39427f1556ae572af5c
---
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/JoinEnum.java
1 file changed, 12 insertions(+), 2 deletions(-)



  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/10/17810/1

diff --git 
a/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/JoinEnum.java
 
b/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/JoinEnum.java
index 9147184..9d38356 100644
--- 
a/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/JoinEnum.java
+++ 
b/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/JoinEnum.java
@@ -899,9 +899,10 @@
                         limit = 
Integer.parseInt(lop.getMaxObjects().getValue().toString());
                     }
                 }
-            }
-            if (op.getOperatorTag() == LogicalOperatorTag.ORDER) {
+            } else if (op.getOperatorTag() == LogicalOperatorTag.ORDER) {
                 return -1; // This is because we cant reduce the selectivity 
of a scan operator when an order by is present.
+            } else if (op.getOperatorTag() == LogicalOperatorTag.GROUP) {
+                return -1; // This is because we cant reduce the selectivity 
of a scan operator when a group by is present.
             }
             op = op.getInputs().get(0).getValue();
         }

--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17810
To unsubscribe, or for help writing mail filters, visit 
https://asterix-gerrit.ics.uci.edu/settings

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Change-Id: Ifd03a57f893e7c22ee3bc39427f1556ae572af5c
Gerrit-Change-Number: 17810
Gerrit-PatchSet: 1
Gerrit-Owner: [email protected]
Gerrit-MessageType: newchange

Reply via email to