>From Vijay Sarathy <[email protected]>: Vijay Sarathy has submitted this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17790 )
Change subject: [ASTERIXDB-3261][COMP] Limit Costing Improvement ...................................................................... [ASTERIXDB-3261][COMP] Limit Costing Improvement Change-Id: Ic6f005778c4c7005d0ac02d51c971f1e87566f80 Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17790 Integration-Tests: Jenkins <[email protected]> Tested-by: Jenkins <[email protected]> Reviewed-by: Vijay Sarathy <[email protected]> --- M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/JoinEnum.java 1 file changed, 14 insertions(+), 1 deletion(-) Approvals: Vijay Sarathy: Looks good to me, approved Jenkins: Verified; Verified 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 edc3251..ee9ec4f 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 @@ -898,7 +898,7 @@ LimitOperator lop = (LimitOperator) op; ILogicalExpression expr = lop.getMaxObjects().getValue(); if (expr != null) { - if (expr.getExpressionTag() != LogicalExpressionTag.VARIABLE) { // must be a constant + if (expr.getExpressionTag() == LogicalExpressionTag.CONSTANT) { // must be a constant return Integer.parseInt(lop.getMaxObjects().getValue().toString()); } } -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17790 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: Ic6f005778c4c7005d0ac02d51c971f1e87566f80 Gerrit-Change-Number: 17790 Gerrit-PatchSet: 2 Gerrit-Owner: [email protected] Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Vijay Sarathy <[email protected]> Gerrit-MessageType: merged
