alamb commented on issue #4628:
URL:
https://github.com/apache/arrow-datafusion/issues/4628#issuecomment-1454796689
> How would OptimizerRule look after the change?
Maybe we could change the signature to something like
```rust
enum OptimizedPlan {
// Optimizer did not make any changes to the original input pla
NoChange(LogicalPlan),
/// Optimizer rewrote the original plan
Rewritten(LogicalPlan),
}
```
/// Try and rewrite `plan` to an optimized form
fn try_optimize(
&self,
plan: &LogicalPlan,
config: &dyn OptimizerConfig,
) -> Result<OptimizedPlan>;
--
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]
