mihaibudiu commented on code in PR #4654:
URL: https://github.com/apache/calcite/pull/4654#discussion_r2582583512
##########
core/src/main/java/org/apache/calcite/sql2rel/RelDecorrelator.java:
##########
@@ -1502,10 +1502,12 @@ private static void findCorrelationEquivalent(CorRef
correlation, RexNode e)
case EQUALS:
final RexCall call = (RexCall) e;
final List<RexNode> operands = call.getOperands();
- if (references(operands.get(0), correlation)) {
+ if (!RexUtil.containsFieldAccess(operands.get(1))
Review Comment:
I think that the JavaDoc of this function needs to be changed; it no longer
reflects what it is doing.
--
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]