indigophox commented on PR #34817: URL: https://github.com/apache/arrow/pull/34817#issuecomment-1906939773
Not thinking about the DoExchange case yet, but as far as DoAction, I believe that (I'm assuming we're talking about also eliminating the implicit callback at :515 for sanity?) moving the SendingHeaders call to after the handler call at :522 is exactly the solution needed. I looked into the call path down from :515 and there's a lot of fanout and so far I'm not following the semantics perfectly—I can spend some more time on that unless you have some minutes to do the chainsaw work there and see if it flies? I can rebase against an experimental commit of the and confirm whether it makes the middleware callback order correct (as I fully expect it would, but just to be sure..). Thanks! -- 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]
