WillAyd commented on PR #1093: URL: https://github.com/apache/arrow-adbc/pull/1093#issuecomment-1760455966
OK after implemented the CopyWriter separately this should work; need to figure out the CI build failure which I'm not getting locally, but think will be green after that. From basic benchmarking in #1189 I don't know that this is any faster than what we have now, but this is also a pretty inefficient implementation. I assume we can speed things up a lot by pre-allocating the write buffer or reserving space rather than leaving it to individual BufferAppend calls to do that -- 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]
