Dandandan commented on a change in pull request #1183:
URL: https://github.com/apache/arrow-datafusion/pull/1183#discussion_r738559693
##########
File path: datafusion/src/execution/context.rs
##########
@@ -176,6 +176,7 @@ impl ExecutionContext {
config,
execution_props: ExecutionProps::new(),
object_store_registry: Arc::new(ObjectStoreRegistry::new()),
+ has_optimized: false,
Review comment:
If we want to do this, I think it should be something on the
`LogicalPlan` instead.
On the other hand, I don't think double optimize on different query
executions (e.g. collect, show) is something very beneficial.
The slower part of optimization is collecting statistics and using it for
cost based optimizations and pruning, which is not something we do in the
logical optimizations.
--
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]
