snuyanzin commented on code in PR #27287:
URL: https://github.com/apache/flink/pull/27287#discussion_r2572828980
##########
flink-table/flink-table-planner/src/main/java/org/apache/flink/table/planner/functions/inference/TypeInferenceOperandInference.java:
##########
@@ -81,7 +80,7 @@ public void inferOperandTypes(
false)) {
inferOperandTypesOrError(unwrapTypeFactory(callBinding),
callContext, operandTypes);
}
- } catch (ValidationException | CalciteContextException e) {
+ } catch (ValidationException e) {
// let operand checker fail
Review Comment:
rethrowing here as ``` will not help for complex cases and will leads again
to `StackOverflow`: first it rethrows as `ValidationException` and then
swallows.
For that reason there is `testNestedCoalesceOnInvalidField` highlighting
this case with a query
```sql
SELECT coalesce(SELECT coalesce(SELECT coalesce(invalid)))
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
