Shockang commented on pull request #33101: URL: https://github.com/apache/spark/pull/33101#issuecomment-874215443
> Looks fine. The tests may be overkill but OK :) > Are there any other instance of .mkdirs() out there that should use the utility method, while we're here? Thank you for your attention (｡･∀･)ﾉﾞ In the Spark source code, there are two user-facing pieces of code and more than 30 test cases using mkdirs. Do you mean I need to replace all these places? -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
