mingmwang commented on pull request #2029: URL: https://github.com/apache/arrow-datafusion/pull/2029#issuecomment-1073414875
> Thanks @mingmwang > > I went over this PR carefully and it looks like a great step forward to me. Thank you > > The use of `Mutex` around the `SessionState` feels unecessary to me, but I also think we can remove it as a follow on PR Hi, @alamb The use of `Mutex` around the `SessionState` is because we want to allow users to change Session related configurations on the fly. Users can use 'Set' command or use 'settings' in the 'ExecuteQueryParams' to reset the session configurations, and at the same time there might be some queries still running with the old configuration. -- 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]
