>From Wail Alkowaileet <[email protected]>:

Wail Alkowaileet has submitted this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/18015 )

Change subject: [NO ISSUE][COMP] Include RUNNINGAGGREGATE & INTERSECT as scoped 
operators
......................................................................

[NO ISSUE][COMP] Include RUNNINGAGGREGATE & INTERSECT as scoped operators

- user model changes: no
- storage format changes: no
- interface changes: no

Details:
Include RUNNINGAGGREGATE & INTERSECT as scoped operators in
pushdown context.

Change-Id: I3a7b316d2bd04c64f0e328e7077a75eb784c4eb1
Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/18015
Integration-Tests: Jenkins <[email protected]>
Tested-by: Jenkins <[email protected]>
Reviewed-by: Wail Alkowaileet <[email protected]>
Reviewed-by: Ali Alsuliman <[email protected]>
---
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/pushdown/PushdownContext.java
1 file changed, 24 insertions(+), 1 deletion(-)

Approvals:
  Wail Alkowaileet: Looks good to me, but someone else must approve
  Ali Alsuliman: Looks good to me, approved
  Jenkins: Verified; Verified

Objections:
  Anon. E. Moose #1000171: Violations found




diff --git 
a/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/pushdown/PushdownContext.java
 
b/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/pushdown/PushdownContext.java
index 51d25a4..e565b8d 100644
--- 
a/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/pushdown/PushdownContext.java
+++ 
b/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/pushdown/PushdownContext.java
@@ -244,7 +244,8 @@

     private static Set<LogicalOperatorTag> getScopeOperators() {
         return EnumSet.of(LogicalOperatorTag.INNERJOIN, 
LogicalOperatorTag.LEFTOUTERJOIN, LogicalOperatorTag.GROUP,
-                LogicalOperatorTag.AGGREGATE, LogicalOperatorTag.WINDOW, 
LogicalOperatorTag.UNIONALL);
+                LogicalOperatorTag.AGGREGATE, LogicalOperatorTag.WINDOW, 
LogicalOperatorTag.RUNNINGAGGREGATE,
+                LogicalOperatorTag.UNIONALL, LogicalOperatorTag.INTERSECT);
     }

 }

--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/18015
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: I3a7b316d2bd04c64f0e328e7077a75eb784c4eb1
Gerrit-Change-Number: 18015
Gerrit-PatchSet: 2
Gerrit-Owner: Wail Alkowaileet <[email protected]>
Gerrit-Reviewer: Ali Alsuliman <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Wail Alkowaileet <[email protected]>
Gerrit-MessageType: merged

Reply via email to