liyafan82 commented on a change in pull request #1910: [CALCITE-3915] Add rule
listener to report rule attempts and time at …
URL: https://github.com/apache/calcite/pull/1910#discussion_r407305137
##########
File path:
core/src/main/java/org/apache/calcite/plan/volcano/VolcanoPlanner.java
##########
@@ -167,7 +137,13 @@
/**
* Listener for this planner, or null if none set.
*/
- RelOptListener listener;
+ MulticastRelOptListener listener;
+
+ /**
+ * Listener for couting the attempts of each rule.
+ * Only enabled under DEBUG.
+ */
+ RuleAttemptsListener ruleAttemptsListener;
Review comment:
It seems this field is never initialized?
----------------------------------------------------------------
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
