lhotari commented on PR #24704: URL: https://github.com/apache/pulsar/pull/24704#issuecomment-3270508561
> LGTM, just have one question: > > for an async function, and it will call the `context.newOutputMessage` method, how to make the `newOutputMessage` method to use the right transaction? shall we create a new method `newOutputMessage` and make it accept a transaction as an argument? @jiangpengcheng The async function doesn't have access to the Transaction in managed transactions. I think that the context must have a reference so that any new messages would get added to the active transaction for that asynchronous call chain. -- 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]
