[CALCITE-4877] Make the exception information of class not found more explicit

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: avatica-1.20.0
• Component/s: avatica
• Labels: pull-request-available
• Assignee: sunjincheng
• Reporter: sunjincheng
• Created: 05/Nov/21 23:04
• Updated: 14/Dec/21 00:06
• Resolved: 07/Nov/21 09:28

Description

Make the exception information of class not found more explicit when we want REATE SCHEMA or CREATE TABLE supported.
when we not add the class to the class path we got the exception as follows:

Error: Error while executing SQL "CREATE TABLE t (i INTEGER, j VARCHAR(10))": Property 'org.apache.calcite.sql.parser.ddl.SqlDdlParserImpl#FACTORY' not valid for plugin type org.apache.calcite.sql.parser.SqlParserImplFactory (state=,code=0)

And would be great if we have a little bit improvement as follows:

 would be great if we can make the error message more explicit , such as: "Property 'org.apache.calcite.sql.parser.ddl.SqlDdlParserImpl#FACTORY' not valid as 'org.apache.calcite.sql.parser.ddl.SqlDdlParserImpl' not found in the classpath."  

What do you think？

Issue Links

• relates to: CALCITE-4937 Upgrade Calcite to Avatica 1.20
• links to: GitHub Pull Request #161
• links to: GitHub Pull Request #2600
• links to: GitHub Pull Request #2608

Sub-Tasks

• Re-Open the Travis job that uses Calcite master until [CALCITE-4877] is fixed: Open