alamb opened a new pull request #1981:
URL: https://github.com/apache/arrow-datafusion/pull/1981


   # Rationale
   Re https://github.com/apache/arrow-datafusion/issues/1469
   Follow on to https://github.com/apache/arrow-datafusion/pull/1945# from 
@doki23  ❤️ 
   
   # Changes
   Remove some more unnecessary `clone()` while building up `Exprs` from 
`SQLExpr`. 
   
   I wanted a small vanity project to make some actual code changes rather than 
filing ticket or reviewing more PRs lol -- see 
https://github.com/apache/arrow-datafusion/pull/1945#pullrequestreview-903442421
   
   I'll highlight the places below that this isn't just mechanical
   


-- 
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]


Reply via email to