[GitHub] [calcite] chunweilei commented on a change in pull request #2011: [CALCITE-4016] Support trait propagation for EnumerableCalc

[GitBox <gi...@apache.org>]

chunweilei commented on a change in pull request #2011:
URL: https://github.com/apache/calcite/pull/2011#discussion_r437119231



##########
File path: core/src/main/java/org/apache/calcite/plan/RelOptUtil.java
##########
@@ -3766,6 +3771,115 @@ private static void fix(List<RexNode> operands, int before, int after) {
     }
   }
 
+  /**
+   * Gets a shuttle to expand RexLocalRef.
+   */
+  public static RexShuttle getExpandShuttle(RexProgram rexProgram) {
+    return new RexShuttle() {
+      @Override public RexNode visitLocalRef(RexLocalRef localRef) {
+        return rexProgram.expandLocalRef(localRef);
+      }
+    };
+  }
+
+  /**
+   * Determine whether there is mapping between project input and output fields.
+   * Bail out if sort relies on non-trivial expressions.
+   */
+  private static boolean isCollationOnTrivialExpr(
+      List<RexNode> projects,RelDataTypeFactory typeFactory,
+      Mappings.TargetMapping map, RelFieldCollation fc, boolean passDown) {
+    final int index = fc.getFieldIndex();
+    int target = map.getTargetOpt(index);

Review comment:
       I am not sure whether RelOptUtil is the best place to hold these methods. Any suggestion is appreciated.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org