goingforstudying-ctrl commented on PR #42629: URL: https://github.com/apache/superset/pull/42629#issuecomment-5245718825
Good catch, thanks. The request-context branch now tags the call with a per-call session token as well (new _request_tool_call_context in auth.py), so request-backed calls get isolated sessions like every other path instead of sharing the greenlet-scoped one. Since no app context is pushed in that branch, the per-call session is deregistered explicitly on exit, leaving the request's own session to the normal request lifecycle. Pushed as 9b73ab43, with a test covering concurrent tool calls under an active request context plus registry cleanup on exit. The full mcp_service unit test module passes locally. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
