andygrove commented on pull request #8018: URL: https://github.com/apache/arrow/pull/8018#issuecomment-678281123
I had another thought on this. We do need the `ExecutionContextState` to be mutable, but only up until execution begins. Perhaps we should rename `ExecutionContextState` to `MutableExecutionContextState` and create a new `ExecutionContextState` that does not have mutexes and just contains a read-only copy of state that is needed during execution? ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
