[GitHub] [flink] chucheng92 commented on a diff in pull request #22215: [FLINK-31507][table] Move execution logic of ShowOperation out from TableEnvironmentImpl

["chucheng92 (via GitHub)" <gi...@apache.org>]

chucheng92 commented on code in PR #22215:
URL: https://github.com/apache/flink/pull/22215#discussion_r1141566179


##########
flink-table/flink-sql-gateway/src/test/resources/sql/table.q:
##########
@@ -31,12 +31,12 @@ org.apache.flink.table.api.ValidationException: Table with identifier 'default_c
 
 describe non_exist;
 !output
-org.apache.flink.table.api.ValidationException: Tables or views with the identifier 'default_catalog.default_database.non_exist' doesn't exist
+org.apache.flink.table.api.ValidationException: Tables or views with the identifier 'default_catalog.default_database.non_exist' doesn't exist.
 !error
 
 desc non_exist;
 !output
-org.apache.flink.table.api.ValidationException: Tables or views with the identifier 'default_catalog.default_database.non_exist' doesn't exist
+org.apache.flink.table.api.ValidationException: Tables or views with the identifier 'default_catalog.default_database.non_exist' doesn't exist.

Review Comment:
   +1 flink-sql-client-test need to add this either. then CliClientITCase will call to run it.



-- 
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: issues-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org