snuyanzin commented on code in PR #27003:
URL: https://github.com/apache/flink/pull/27003#discussion_r2359508404
##########
flink-table/flink-table-api-java/src/main/java/org/apache/flink/table/catalog/CatalogManager.java:
##########
@@ -708,7 +708,7 @@ public ContextResolvedTable
getTableOrError(ObjectIdentifier objectIdentifier) {
return getTable(objectIdentifier)
.orElseThrow(
() ->
- new TableException(
+ new ValidationException(
Review Comment:
a comment about `TableException`
https://github.com/apache/flink/blob/d08abb7b1697a3ba78ee0bc5d31f841eb0c9d386/flink-table/flink-table-common/src/main/java/org/apache/flink/table/api/TableException.java#L24-L29
table not exists is a user error since a user supplies a table name to get.
If a user supplied a name for not existed table then it is a user error
--
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]
