yaooqinn commented on pull request #28738: URL: https://github.com/apache/spark/pull/28738#issuecomment-639524059
> Nice cleanup of the Thriftserver tests! > I think it still has some merit to test the Thriftserver started with the start-thriftserver.sh script, so I would not remove that old infra completely, but maybe leave some basic suite for that? `start-thriftserver.sh` script with port retry is flaky for testing, maybe we can dockerize them with integration test? ---------------------------------------------------------------- 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]
