ulysses-you commented on a change in pull request #30897:
URL: https://github.com/apache/spark/pull/30897#discussion_r547615586



##########
File path: 
sql/hive-thriftserver/src/test/scala/org/apache/spark/sql/hive/thriftserver/ThriftServerWithSparkContextSuite.scala
##########
@@ -102,14 +102,14 @@ trait ThriftServerWithSparkContextSuite extends 
SharedThriftServer {
 
       spark.sparkContext.addSparkListener(listener)
       try {
-        statement.execute(s"SET ${SQLConf.THRIFTSERVER_QUERY_TIMEOUT.key}=1")
+        statement.execute(s"SET ${SQLConf.THRIFTSERVER_QUERY_TIMEOUT.key}=500")

Review comment:
       Sorry I misread by the error msg.
   
   The timeout error msg is confused since the timeout config 
`spark.sql.thriftServer.queryTimeout` can't change the 
`Statement.queryTimeout`, then the error msg is always `Query timed out after 0 
seconds`.
   
   Back to the flaky test, seems the set config sql will timeout sometime if we 
limit the queryTime to 1 second.




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



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to