gabrielenizzoli opened a new pull request #31689: URL: https://github.com/apache/spark/pull/31689
### What changes were proposed in this pull request? In the `SQLConf` object, the `sqlConfEntries` map is globally synchronized (it is a Java `Collections.synchronizedMap`): any operation, including a get, will need to acquire the lock. To avoid having multiple tasks locking on this, a better approach would be to use a map that does not lock on read (like a `ConcurrentHashMap`). ### Why are the changes needed? Multiple tasks performing something as easy a `DataType.sameType(...)` would be locking on the global `sqlConfEntries` lock of the `Collections.synchronizedMap`. ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? No functionality change. Existing unit tests run normally. ---------------------------------------------------------------- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
