indigophox commented on PR #34817: URL: https://github.com/apache/arrow/pull/34817#issuecomment-1907113721
> I am lost. You would stop calling SendingHeaders where it was previously called, and start calling it explicitly in each RPC handler at an appropriate time. I don't see why empty streams need special treatment. Just looking at where you said to do it around L528 here https://github.com/apache/arrow/blob/3fe598ae4dfd7805ab05452dd5ed4b0d6c97d8d5/cpp/src/arrow/flight/transport/grpc/grpc_server.cc#L513-L544 That creates a special case for DoAction getting an empty stream back from the Action handler, so I'm assuming it would be L524 instead... -- 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]
