alamb commented on pull request #10049: URL: https://github.com/apache/arrow/pull/10049#issuecomment-820670195
> One potential user-friendliness thing: would it be worth having SQLMetric use atomics internally to avoid the Mutex<SQLMetric> that'll probably be required for most usage? Almost certainly makes no difference from a perf perspective, the contention will be pretty much non-existent, just thinking about the API. I think this would be a good idea @returnString FYI. Perhaps part of a subsequent PR -- 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]
