klion26 commented on PR #8768: URL: https://github.com/apache/arrow-rs/pull/8768#issuecomment-3480037323
@mbrobbel Thanks for the quick review. As the primitive builder for `VariantToArrow` now only left `Utf8` and `Binary`(all the other types have been implemented, `list/object` will be implemented in other places), I've appended `binary` case to this pr, sorry for any inconvenience this may cause. (pushed to the remote branch before I noticed the review) Appended two commits - one to enhance the test name for Utf8/LargeUtf8 - the other one wants to support the `DataType::Binary` -- 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]
