commit 9592afb63bef3c0b6929f956de2a12a2fc4de05b
author murali4104 <murali.krishna@couchbase.com> Tue Sep 26 09:12:46 2023 -0700
committer Vijay Sarathy <vijay.sarathy@couchbase.com> Tue Sep 26 17:57:39 2023 +0000
tree b4dd7f3ee3be3ecdac71022d7896989be6634ff9
parent d755aef8c8a6efc1a56705c2a6cc1ec05fec444e

[ASTERIXDB-3261][COMP] Limit Costing

Change-Id: Ifd03a57f893e7c22ee3bc39427f1556ae572af5c
Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17810
Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Reviewed-by: Vijay Sarathy <vijay.sarathy@couchbase.com>

asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/JoinEnum.java

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();
         }