(asterixdb) branch master updated: [ASTERIXDB-3531][COMP] Format AssignRuntimeFactory, EmbedProjectRule

[alsuliman]

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 5ff8ab05cd [ASTERIXDB-3531][COMP] Format AssignRuntimeFactory, 
EmbedProjectRule
5ff8ab05cd is described below

commit 5ff8ab05cda106b80ed9eea3fa7d1bd721dbd211
Author: preetham0202 <preetham.polupar...@couchbase.com>
AuthorDate: Thu Dec 12 17:25:36 2024 +0530

    [ASTERIXDB-3531][COMP] Format AssignRuntimeFactory, EmbedProjectRule
    
    Ext-ref: MB-57625
    Change-Id: I9507dd68253bb10381a0f6476666ed5fa7f2ab74
    Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19192
    Integration-Tests: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
    Reviewed-by: Ali Alsuliman <ali.al.solai...@gmail.com>
    Tested-by: Ali Alsuliman <ali.al.solai...@gmail.com>
---
 .../hyracks/algebricks/rewriter/rules/EmbedProjectRule.java | 13 ++++++-------
 .../runtime/operators/std/AssignRuntimeFactory.java         |  4 +++-
 2 files changed, 9 insertions(+), 8 deletions(-)

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 acb2f620e9..4acb509b5e 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 @@ import 
org.apache.hyracks.algebricks.core.rewriter.base.IAlgebraicRewriteRule;
  */
 
 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 @@ public class EmbedProjectRule implements 
IAlgebraicRewriteRule {
         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 a39cd05341..6cd7a05582 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 @@ public class AssignRuntimeFactory extends 
AbstractOneInputOneOutputRuntimeFactor
             }
             sb.append(evalFactories[i].toString());
         }
-        sb.append("]");
+        sb.append("] ");
+        sb.append(" project: ");
+        sb.append(Arrays.toString(projectionList));
         return sb.toString();
     }