alamb commented on PR #8260: URL: https://github.com/apache/arrow-rs/pull/8260#issuecomment-3308644760
> I think it's better way to expose 'ArrowRowGroupWriter' and add function `into_writers` > `ArrowRowGroupWriter` it's helpers structure for use `Vec<ArrowColumnWriter>` Sorry I put a response to this on a different PR: https://github.com/apache/arrow-rs/pull/8162#discussion_r2360358117 Basically, I am not sure that `ArrowRowGroupWriter` adds a lot of value so I am not sure it needs to be pub. Maybe we can make an documentation example showing how parallel writing to row groups / column chunks would look if we made the `ArrowRowGroupWriter` pub 🤔. Given how much effort we go through in arrow to keep the API stable, I am hesitant to add anything more to the API than necessary. I think we can make `ArrowRowGroupWriter` as a follow on PR / in a follow on release too. The code in https://github.com/apache/arrow-rs/pull/8162 doesn't preclude its use -- 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]
