robalar commented on issue #642: URL: https://github.com/apache/arrow-rs-object-store/issues/642#issuecomment-3907769545
@tustvold, yeah, the problem being that we already have exisiting implementations for our tracing that wrap reqwests `RequestBuilder`, so this is the type I want access to. If I were just to wrap the existing `HttpConnector`, I'd just have access to object stores `HttpRequest` - which I would need to add the same tracing to (which I _think_ would be do-able, just need to be able to set headers and see errors etc). However, having access to this methods would be the easiest and reduce maintanence burden for us, but you are obviously increasing your public surface area so its an increased burden for you. Let me know what you think! -- 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]
