HeartSaVioR commented on issue #27001: [SPARK-30345][SQL] Fix intermittent test 
failure (ConnectException) on 
ThriftServerQueryTestSuite/ThriftServerWithSparkContextSuite
URL: https://github.com/apache/spark/pull/27001#issuecomment-569201728
 
 
   > The fix itself LGTM, but do you know why Thrift(Binary|Http)CLIService 
prepare and launch the service asynchronously? Seems end-users can also hit 
this problem if they run queries right after the server is started.
   
   I don't know about history, but if I get the commit history correctly, 
codebase of `sql/hive-thiftserver` just came from Hive, and we just fixed 
minors. So that's basically inherited from Hive.
   
   Here we're embedding Hive server in test suites hence get a chance to 
encounter race condition, but most likely it will run as standalone app which 
end users would just think server is not ready and rerun the query when query 
fails for such reason. 
   
   Btw, there's another test suites which runs Hive server as external 
standalone app - it shouldn't encounter this issue but it would show longer 
startup and shutdown. 
   
   
https://github.com/apache/spark/blob/9c046dc8084500860f3820bdbd2be04a6e491431/sql/hive-thriftserver/src/test/scala/org/apache/spark/sql/hive/thriftserver/HiveThriftServer2Suites.scala#L986-L1215

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


With regards,
Apache Git Services

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

Reply via email to