[asterixdb] branch master updated: [ASTERIXDB-3261][COMP] Limit Costing

[alsuliman]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 9592afb63b [ASTERIXDB-3261][COMP] Limit Costing
9592afb63b is described below

commit 9592afb63bef3c0b6929f956de2a12a2fc4de05b
Author: murali4104 <murali.kris...@couchbase.com>
AuthorDate: Tue Sep 26 09:12:46 2023 -0700

    [ASTERIXDB-3261][COMP] Limit Costing
    
    Change-Id: Ifd03a57f893e7c22ee3bc39427f1556ae572af5c
    Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17810
    Integration-Tests: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
    Tested-by: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
    Reviewed-by: Vijay Sarathy <vijay.sara...@couchbase.com>
---
 .../main/java/org/apache/asterix/optimizer/rules/cbo/JoinEnum.java   | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

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 9147184418..9d38356bb9 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 @@ public class JoinEnum {
                         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();
         }