[CALCITE-2003] Avoid global synchronization around openConnection in JdbcMeta

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: avatica-1.11.0
• Component/s: avatica
• Assignee: Josh Elser
• Reporter: Josh Elser
• Created: 06/Oct/17 17:17
• Updated: 09/Mar/18 23:25
• Resolved: 09/Oct/17 16:07

Description

Had a report from a user where opening new connections to Avatica were stuck. Upon investigation, there was a single thread which had grabbed the local on the JdbcMeta instance in JdbcMeta#openConnection(ConnectionHandle), but was not returning out of the DriverManager.getConnection(...) call.

Using the ConcurrentMap API over the Guava Cache API gives us enough flexibility that we can handle this safely without the total synchronization.

Issue Links

• links to: GitHub Pull Request #17