houqp opened a new issue #593:
URL: https://github.com/apache/arrow-datafusion/issues/593
**Is your feature request related to a problem or challenge? Please describe
what you are trying to do.**
```
pub fn from(plan: &LogicalPlan) -> Self {
Self { plan: plan.clone() }
```
The from method takes a LogicalPlan reference, then does clone internally.
**Describe the solution you'd like**
It would be better to take an owned value and let the caller handle the
clone if needed.
--
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:
us...@infra.apache.org
