mapleFU commented on PR #46372: URL: https://github.com/apache/arrow/pull/46372#issuecomment-2879251495
@emkornfield @wgtmac @pitrou @zeroshade This patch add some basic variant decoding tools. Some thoughts: How would the interface for visiting variant like? The simplist way is cast `<metadata, value>` pairs to a `ptr<::arrow::Scalar>`, but this is too slow and needs to read whole data. We can also wraps a `std::variant`, but I think it's also slow and needs to dynamic dispatch the visitor. Here I just add visitor for every type. And currently, `getInt64` would only supports read int64. Any idea is welcome -- 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]
