[CALCITE-4591] RelRunner should throw SQLException if prepare fails

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.27.0
• Assignee: Julian Hyde
• Reporter: Julian Hyde
• Created: 26/Apr/21 18:35
• Updated: 03/Jun/21 22:32
• Resolved: 27/Apr/21 02:41

Description

interface RelRunner allows statements to be prepared from a RelNode tree rather than via SQL. A RelRunner is typically obtained from a JDBC connection, by calling connection.unwrap(RelRunner.class). As such, RelRunner is an extension to the JDBC driver, and therefore the RelRunner.prepare(RelNode) method should throw SQLException, rather than RuntimeException as it does today.

This change adds a new method

PreparedStatement prepareStatement(RelNode) throws SQLException

and marks the current prepare method as deprecated. (As RelRunner is marked "experimental", we could have removed or changed the old method without notice, but chose not to.)

Issue Links

• relates to: CALCITE-4585 Improve error message from RelRunner