chebbyChefNEQ commented on issue #6469: URL: https://github.com/apache/arrow-rs/issues/6469#issuecomment-2379306871
Hi yeah I totally get both points: 1) store should be created once and cached, and 2) making builder async is going to be a major API change. Unfortunately :( our code makes a bit hard to do DI style constructions and we can't easily cache the store handle always. Also `reqwest` seems not have blocking IO behavior in its builder with other tls features. `rustls-tls-native-roots` seems like the only with with this issue. I def lack the context on what we need native roots maybe that's something we could change? I guess another thing is having blocking IO here blocks our tokio worker and that causes some issues, albeit minor most of the time. -- 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]
