LantaoJin opened a new pull request #28385:
URL: https://github.com/apache/spark/pull/28385


   ### What changes were proposed in this pull request?
   In our production, we find that many shuffle files could be located in
   
/hadoop/2/yarn/local/usercache/b_carmel/appcache/application_1586487864336_4602/*null*-107d4e9c-d3c7-419e-9743-a21dc4eaeb3f/3a
   
   The Util.createDirectory() uses a default parameter "spark"
   ```
     def createDirectory(root: String, namePrefix: String = "spark"): File
   ```
   But in some cases, the actual namePrefix is null. If the method is called 
with null, then the default value would not be applied.
   
   ### Does this PR introduce any user-facing change?
   No
   
   
   ### How was this patch tested?
   Add a UT.
   


----------------------------------------------------------------
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:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to