You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@asterixdb.apache.org by AsterixDB Code Review <do...@asterix-gerrit.ics.uci.edu> on 2024/12/12 11:56:58 UTC
Change in asterixdb[master]: [ASTERIXDB-3531][COMP] Format AssignRuntimeFactory, EmbedProjectRule
From <pr...@gmail.com>:
preethampoluparthi@gmail.com has uploaded this change for review. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19192 )
Change subject: [ASTERIXDB-3531][COMP] Format AssignRuntimeFactory, EmbedProjectRule
......................................................................
[ASTERIXDB-3531][COMP] Format AssignRuntimeFactory, EmbedProjectRule
Change-Id: I9507dd68253bb10381a0f6476666ed5fa7f2ab74
---
M hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/EmbedProjectRule.java
M hyracks-fullstack/algebricks/algebricks-runtime/src/main/java/org/apache/hyracks/algebricks/runtime/operators/std/AssignRuntimeFactory.java
2 files changed, 18 insertions(+), 8 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/92/19192/1
diff --git a/hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/EmbedProjectRule.java b/hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/EmbedProjectRule.java
index acb2f62..4acb509 100644
--- a/hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/EmbedProjectRule.java
+++ b/hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/EmbedProjectRule.java
@@ -34,13 +34,6 @@
*/
public class EmbedProjectRule implements IAlgebraicRewriteRule {
- private static boolean isProjectable(LogicalOperatorTag op) {
- return switch (op) {
- case ASSIGN, UNNEST, LEFT_OUTER_UNNEST, RUNNINGAGGREGATE, SELECT -> true;
- default -> false;
- };
- }
-
@Override
public boolean rewritePost(Mutable<ILogicalOperator> opRef, IOptimizationContext context) {
@@ -65,4 +58,10 @@
return true;
}
+ private static boolean isProjectable(LogicalOperatorTag op) {
+ return switch (op) {
+ case ASSIGN, UNNEST, LEFT_OUTER_UNNEST, RUNNINGAGGREGATE, SELECT -> true;
+ default -> false;
+ };
+ }
}
diff --git a/hyracks-fullstack/algebricks/algebricks-runtime/src/main/java/org/apache/hyracks/algebricks/runtime/operators/std/AssignRuntimeFactory.java b/hyracks-fullstack/algebricks/algebricks-runtime/src/main/java/org/apache/hyracks/algebricks/runtime/operators/std/AssignRuntimeFactory.java
index a39cd05..6cd7a05 100644
--- a/hyracks-fullstack/algebricks/algebricks-runtime/src/main/java/org/apache/hyracks/algebricks/runtime/operators/std/AssignRuntimeFactory.java
+++ b/hyracks-fullstack/algebricks/algebricks-runtime/src/main/java/org/apache/hyracks/algebricks/runtime/operators/std/AssignRuntimeFactory.java
@@ -92,7 +92,9 @@
}
sb.append(evalFactories[i].toString());
}
- sb.append("]");
+ sb.append("] ");
+ sb.append(" project: ");
+ sb.append(Arrays.toString(projectionList));
return sb.toString();
}
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19192
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: I9507dd68253bb10381a0f6476666ed5fa7f2ab74
Gerrit-Change-Number: 19192
Gerrit-PatchSet: 1
Gerrit-Owner: preethampoluparthi@gmail.com
Gerrit-MessageType: newchange