yaooqinn commented on a change in pull request #30643:
URL: https://github.com/apache/spark/pull/30643#discussion_r540654510



##########
File path: 
sql/hive-thriftserver/src/test/scala/org/apache/spark/sql/hive/thriftserver/HiveThriftServer2Suites.scala
##########
@@ -1122,62 +1123,6 @@ object ServerMode extends Enumeration {
   val binary, http = Value
 }
 
-abstract class HiveThriftJdbcTest extends HiveThriftServer2Test {

Review comment:
       Reflecting this is to bring its functions to the superclass, which is 
used to add a probe to check the server is responsive after started. And If It 
is not it can retry to reduce flakiness and dump the server log to 
`unit-test.log` for us to see what is wrong actually




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