[DRILL-8009] DrillConnectionImpl#isValid() doesn't correspond JDBC API

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.20.0
• Component/s: Client - JDBC
• Assignee: Maksym Rymar
• Reporter: Maksym Rymar
• Reviewer: Paul Rogers
• Created: 13/Oct/21 14:52
• Updated: 11/Feb/22 06:41
• Resolved: 10/Feb/22 10:24

Description

DrillConnectionImpl#isValid() doesn't correspond Java JDBC API documentation. Current implementation doesn't do actual connection verify and doesn't cover cases like drillbit termination and network issues.

java.sql.Connection#isValid() (which extends DrillConnectionImpl) widely used in JDBC connection pools like HikariCP and without right implementation, Drill JDBC driver cannot be used with them, because connections will keep alive forever.