danny0405 commented on a change in pull request #2664:
URL: https://github.com/apache/calcite/pull/2664#discussion_r777164619
##########
File path:
core/src/main/java/org/apache/calcite/rel/logical/LogicalCorrelate.java
##########
@@ -86,26 +107,38 @@ public LogicalCorrelate(RelInput input) {
}
/** Creates a LogicalCorrelate. */
- public static LogicalCorrelate create(RelNode left, RelNode right,
+ public static LogicalCorrelate create(RelNode left, RelNode right,
List<RelHint> hints,
CorrelationId correlationId, ImmutableBitSet requiredColumns,
JoinRelType joinType) {
final RelOptCluster cluster = left.getCluster();
final RelTraitSet traitSet = cluster.traitSetOf(Convention.NONE);
- return new LogicalCorrelate(cluster, traitSet, left, right, correlationId,
+ return new LogicalCorrelate(cluster, traitSet, hints, left, right,
correlationId,
requiredColumns, joinType);
}
+ @Deprecated // to be removed before 1.23
+ public static LogicalCorrelate create(RelNode left, RelNode right,
Review comment:
Do you mean 'to be removed before 2.0' ?
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
