Re: [FlightRPC][Java] HTTP cookie support

2020-10-30 Thread David Li
I'd rather people opt in to extra functionality (we don't even require TLS by default, though gRPC itself does in Java). On that note, I see a somewhat related proposal for sessions in Flight; it's worth thinking if that can just be built on top of cookies: https://issues.apache.org/jira/browse/AR

Re: [FlightRPC][Java] HTTP cookie support

2020-10-30 Thread James Duong
Thanks for the feedback David. I'll add a client builder option to make it more convenient to enable. Actually, would it make sense for something like this to be enabled by default? There's no drawback when connecting to FlightServers that don't use cookies other than a bit of overhead searching

Re: [FlightRPC][Java] HTTP cookie support

2020-10-30 Thread David Li
Hey James, I looked over the PR and it seems quite straightforward - thanks for contributing this. Best, David On 10/29/20, James Duong wrote: > Hi all, > > I've created a PR for changing the Java FlightClient to have the ability to > send/receive cookie headers: > https://github.com/apache/arr