davidradl commented on code in PR #26781:
URL: https://github.com/apache/flink/pull/26781#discussion_r2200063187
##########
flink-table/flink-sql-gateway/src/main/java/org/apache/flink/table/gateway/service/operation/OperationExecutor.java:
##########
@@ -193,6 +194,7 @@ public ResultFetcher configureSession(OperationHandle
handle, String statement)
String.format(
"Unsupported statement for configuring
session:%s\n"
+ "The configureSession API only supports
to execute statement of type "
+ + "SET, RESET, "
Review Comment:
nit: could we put all of these values into a variable we just iterate.
Ideally the variable would be derived from what the API uses to validate.
--
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]
