Github user scwf commented on the pull request:
https://github.com/apache/spark/pull/2671#issuecomment-58029661
Hi, @liancheng, thanks for you comments that is very useful.
1 Actually i have tried start a server for a every test case but the second
one failed due to "org.apache.spark.sql.hive.thriftserver.HiveThriftServer2
running as process 82266. Stop it first." Now i get the reason from your patch
"A Thread.sleep has to be introduced because the kill command used in
stop-thriftserver.sh is not synchronous."
2 Use a tail process is ok, actually i think about it. Since the log file
won't be a big file, so here i mike to use ```fileToString```
3 Yeah, here should remove log file
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
