[CALCITE-4628] If SqlImplementor fails, include the RelNode in the exception

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.28.0
• Assignee: Julian Hyde
• Reporter: Julian Hyde
• Created: 01/Jun/21 19:06
• Updated: 20/Oct/21 05:24
• Resolved: 13/Oct/21 23:33

Description

If SqlImplementor fails, include the RelNode in the exception. (For example, we recently encountered a java.lang.ArrayIndexOutOfBoundsException where the problem was in the RelNode but the RelNode was not in the stack.)

Add a catch block that calls Util.throwAsRuntime to the SqlImplementor.visitRoot method. That method seems to be a widely used entry point.