lidavidm commented on PR #34817: URL: https://github.com/apache/arrow/pull/34817#issuecomment-1907090592
> That gets us back to needing middleware to access the session store, which just moves the problem around in order to solve it... I'm thinking about doing the other solution very locally, but the header handling needs to be triggered from within the handler itself (not even DoAction() which can't intercede at the right point), so I will need to add the stream interposer to defaut-trigger the header handling on stream onNext & onError & onCompleted for compatibility with existing implementations... objections? No: just do everything inline in the RPC handler/a helper, no middleware involved at all. You can read and write headers inside the handler. -- 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]
