>From Vijay Sarathy <[email protected]>: Vijay Sarathy has submitted this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17810 )
Change subject: [ASTERIXDB-3261][COMP] Limit Costing ...................................................................... [ASTERIXDB-3261][COMP] Limit Costing Change-Id: Ifd03a57f893e7c22ee3bc39427f1556ae572af5c Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17810 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, 16 insertions(+), 2 deletions(-) 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 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: 2 Gerrit-Owner: [email protected] Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Vijay Sarathy <[email protected]> Gerrit-MessageType: merged
