zhztheplayer commented on a change in pull request #1237: [CALCITE-3095] Add
several system properties to control enabling/disabling of rules and traits
URL: https://github.com/apache/calcite/pull/1237#discussion_r288910198
##########
File path:
core/src/main/java/org/apache/calcite/plan/volcano/VolcanoPlanner.java
##########
@@ -276,9 +264,6 @@ public VolcanoPlanner(RelOptCostFactory costFactory, //
super(costFactory == null ? VolcanoCost.FACTORY : costFactory, //
externalContext);
this.zeroCost = this.costFactory.makeZeroCost();
- // If LOGGER is debug enabled, enable provenance information to be captured
- this.provenanceMap = LOGGER.isDebugEnabled() ? new HashMap<>()
- : Util.blackholeMap();
}
Review comment:
I guess you are able to see such a change ~20 minutes ago in this PR. That's
because I was resolving some conflicts and the PR was not ready at that time.
Now in master branch the changes should be already corrected.
----------------------------------------------------------------
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
