mihaibudiu commented on code in PR #4042:
URL: https://github.com/apache/calcite/pull/4042#discussion_r1834826784
##########
core/src/main/java/org/apache/calcite/sql2rel/RelDecorrelator.java:
##########
@@ -354,26 +376,54 @@ protected RelNode decorrelate(RelNode root) {
private HepPlanner createPlanner(HepProgram program) {
// Create a planner with a hook to update the mapping tables when a
// node is copied when it is registered.
- return new HepPlanner(
- program,
- context,
- true,
- createCopyHook(),
- RelOptCostImpl.FACTORY);
+ HepPlanner planner =
+ new HepPlanner(
+ program,
+ context,
+ true,
+ createCopyHook(),
+ RelOptCostImpl.FACTORY);
+ planner.setDecorrelator(this);
+ return planner;
}
+ /**
+ * Tries to remove the {@link org.apache.calcite.rel.core.Correlate}
instances of a query
Review Comment:
I think "removes some instances of Correlate from a query plan" is a bit
more precise.
--
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]
