lidavidm commented on pull request #10230: URL: https://github.com/apache/arrow/pull/10230#issuecomment-843273563
Just to follow up - APIs like FileSystem return `shared_ptr<OutputStream>` so it would be very annoying to take `unique_ptr`. And we could just take only `OutputStream*` but IMO even if the caller is supposed to keep the pointer alive, it's safer to offer the option to take a `shared_ptr` by default to minimize mistakes. -- 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]
