alamb commented on pull request #8018: URL: https://github.com/apache/arrow/pull/8018#issuecomment-678406791
That definitely sounds cleaner to me On Fri, Aug 21, 2020 at 12:07 PM Andy Grove <[email protected]> wrote: > @alamb <https://github.com/alamb> I think you are right. Rather than > having Arc<Mutex<_>> within ExecutionContextState, we should have > Arc<Mutex<ExecutionContextState>> where we need mutability and then > clone() the state to get a read-only snapshot when we need it for > execution. > > — > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub > <https://github.com/apache/arrow/pull/8018#issuecomment-678369613>, or > unsubscribe > <https://github.com/notifications/unsubscribe-auth/AADXZMNL3AKJJYNL7NO7ZLLSB2LTTANCNFSM4QGRFXAQ> > . > ---------------------------------------------------------------- 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: [email protected]
