attilapiros commented on pull request #31877: URL: https://github.com/apache/spark/pull/31877#issuecomment-801999551
> For the test case concern, I can still remove it but I prefer to have the test coverage for the new function for now. We can revise the test suite later with the `withHttpServer` refactoring. +1 for keeping the test and I will revise it in my PR -- 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]
