HyukjinKwon opened a new pull request #23405: [SPARK-26496][SS][TESTS] Avoid to 
use Random.nextString in StreamingInnerJoinSuite
URL: https://github.com/apache/spark/pull/23405
 
 
   ## What changes were proposed in this pull request?
   
   Similar with https://github.com/apache/spark/pull/21446. Looks random string 
is not quite safe as a directory name.
   
   ```scala
   scala> val prefix = Random.nextString(10); val dir = new File("/tmp", "del_" 
+ prefix + "-" + UUID.randomUUID.toString); dir.mkdirs()
   prefix: String = 窽텘⒘駖ⵚ駢⡞Ρ닋੎
   dir: java.io.File = /tmp/del_窽텘⒘駖ⵚ駢⡞Ρ닋੎-a3f99855-c429-47a0-a108-47bca6905745
   res40: Boolean = false  // nope, didn't like this one
   ```
   
   ## How was this patch tested?
   
   Unit test was added, and manually.
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


With regards,
Apache Git Services

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

Reply via email to