lidavidm commented on PR #43632: URL: https://github.com/apache/arrow/pull/43632#issuecomment-2306060082
> Either remove the extension parameter or change it to be const char* using the metadata encoding like we're doing for the on_error callback I think the metadata encoding is reasonable. I guess it'll force people to do a lot of string parsing, but that's just how it goes. FWIW, do we need to do anything to accommodate the arrow::ArrayStatistics API that @kou has been working on? > Do we want to go ahead with the waker approach? I'm not quite sold on this if we want to have a callback based API. Though I understand wanting to have the most efficient possible theoretical API. At least from my Flight/gRPC based view, though, it seems difficult to take advantage of. -- 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]
