Dandandan edited a comment on pull request #9692: URL: https://github.com/apache/arrow/pull/9692#issuecomment-798772171
I think this makes sense. This is a partial revert from a PR from me (I believe). But I think it is better for "builder" kind of functions where it needs "owned" data. In general, `&Vec` and `&String` are anti patterns though, and some functions taking `Vec` and `String` still could/should be using references instead. ---------------------------------------------------------------- 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]
