GitHub user dongjoon-hyun opened a pull request:

    https://github.com/apache/spark/pull/21446

    [SPARK-19613][SS][TEST] Random.nextString is not safe for directory 
namePrefix

    ## What changes were proposed in this pull request?
    
    `Random.nextString` is good for generating random string data, but it's not 
suitable for directory name prefix. This PR uses more safe directory namePrefix.
    
    ```
    scala> scala.util.Random.nextString(10)
    res0: String = 馨쭔ᎰႻ穚䃈兩㻞藑並
    
    scala> scala.util.Random.nextInt.abs.toString
    res1: String = 1016732284
    ```
    
    ## How was this patch tested?
    
    Pass the Jenkins.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/dongjoon-hyun/spark SPARK-19613

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/21446.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #21446
    
----
commit a46dc70d86a4f3f8387dc7c2125392de5a2cfb5e
Author: Dongjoon Hyun <dongjoon@...>
Date:   2018-05-28T15:53:22Z

    [SPARK-19613][SS][TEST] Random.nextString is not safe for directory 
namePrefix

----


---

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

Reply via email to