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 17f78eab06 [ASTERIXDB-3261][COMP] Limit Costing
17f78eab06 is described below
commit 17f78eab06c3034daa641b63cad1ac6850797a35
Author: murali4104 <[email protected]>
AuthorDate: Mon Sep 25 16:56:56 2023 -0700
[ASTERIXDB-3261][COMP] Limit Costing
Change-Id: I0a3e1be5e134c839168b584a6f41c45682e078ee
Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17808
Integration-Tests: Jenkins <[email protected]>
Tested-by: Jenkins <[email protected]>
Reviewed-by: Vijay Sarathy <[email protected]>
---
.../java/org/apache/asterix/optimizer/rules/cbo/JoinEnum.java | 8 ++++++--
1 file changed, 6 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 752064a6c3..9147184418 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
@@ -889,19 +889,23 @@ public class JoinEnum {
private int findLimitValue(AbstractLogicalOperator oper) {
ILogicalOperator op = oper;
+ int limit = -1;
while (op.getOperatorTag() != LogicalOperatorTag.EMPTYTUPLESOURCE) {
if (op.getOperatorTag() == LogicalOperatorTag.LIMIT) {
LimitOperator lop = (LimitOperator) op;
ILogicalExpression expr = lop.getMaxObjects().getValue();
if (expr != null) {
if (expr.getExpressionTag() ==
LogicalExpressionTag.CONSTANT) { // must be a constant
- return
Integer.parseInt(lop.getMaxObjects().getValue().toString());
+ limit =
Integer.parseInt(lop.getMaxObjects().getValue().toString());
}
}
}
+ 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.
+ }
op = op.getInputs().get(0).getValue();
}
- return -1;
+ return limit;
}
private boolean isPredicateCardinalityAnnotationPresent(ILogicalExpression
leExpr) {
