leoyvens commented on PR #5683: URL: https://github.com/apache/arrow-rs/pull/5683#issuecomment-2075300691
For arrow-flight it is truly only required for the high-level client, not for the server since `FlightServiceServer` is just a `Service` trait implementation. But I'm happy to rename the feature gate. My use-case would be to build a simple server by using `hyper` directly, without axum or the tonic server. I wanted to use hyper 1.0 without waiting for tonic to upgrade, but turns out this isn't possible due to conflicting `http` crate versions between hyper 1.0 and tonic. That is say, I have not been able to get my use case for this PR actually working. And once tonic upgrades to hyper 1.0 and http 1.0, I suppose the only uneeded dependency would be axum itself, and I wouldn't classify that as a 'real pain'. So I think it's fair if you'd rather not do this at this point. -- 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]
