hsyuan commented on a change in pull request #1468: [CALCITE-3366]
RelDecorrelator supports Union
URL: https://github.com/apache/calcite/pull/1468#discussion_r326951042
##########
File path: core/src/main/java/org/apache/calcite/sql2rel/RelDecorrelator.java
##########
@@ -1242,6 +1244,37 @@ public Frame decorrelateRel(Join rel) {
return register(rel, newJoin, mapOldToNewOutputs, corDefOutputs);
}
+ public Frame decorrelateRel(LogicalUnion rel) {
+ return decorrelateRel((Union) rel);
+ }
Review comment:
This is not needed, which can be covered by Union. Other operator has method
for logical operator just for legacy reasons.
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
