waynexia commented on PR #4612: URL: https://github.com/apache/arrow-datafusion/pull/4612#issuecomment-1351075714
> I wonder if perhaps this is a sign that we don't really need these locks 🤔 will have a play around IMO that lock cannot be avoided. If we don't put it over `SessionState` then we need to either put it in the `SessionState` or change all the methods to require `&mut self`, which is likely to be put inside another lock to achieve the "interior mutability" 🥲 -- 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]
