fsk119 commented on a change in pull request #15197:
URL: https://github.com/apache/flink/pull/15197#discussion_r594834367



##########
File path: flink-table/flink-sql-client/src/test/resources/sql/set.q
##########
@@ -42,20 +42,11 @@ CREATE TABLE hive_table (
 [INFO] Table has been created.
 !info
 
-# list the configured configuration
-set;
-table.sql-dialect=hive
-!ok
-

Review comment:
       Currently the set command will print all the commands in the 
configuration, e.g. 
   
   ```
   execution.attached=true
   execution.runtime-mode=batch
   execution.savepoint.ignore-unclaimed-state=false
   execution.shutdown-on-attached-exit=false
   execution.target=remote
   jobmanager.rpc.address=localhost
   pipeline.classpaths=
   
pipeline.jars=file:/var/folders/1k/kcnx0zh91c1dkl0_m00ds2mh0000gp/T/junit3565817602485898043/test-jar/test-classloader-udf.jar
   rest.port=63196
   sql-client.execution.result-mode=tableau
   [DEPRECATED] execution.result-mode=tableau
   [DEPRECATED] execution.type=batch
   ```
   
   In the test, we need to use jar the location of the jar is random. 
Therefore, I remove the test about the set. 
   




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to