Github user vanzin commented on a diff in the pull request:

    https://github.com/apache/spark/pull/22504#discussion_r226740562
  
    --- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala ---
    @@ -2863,6 +2870,14 @@ private[spark] object Utils extends Logging {
       def stringHalfWidth(str: String): Int = {
         if (str == null) 0 else str.length + fullWidthRegex.findAllIn(str).size
       }
    +
    +  private[spark] def sanitizeDirName(str: String): String = {
    --- End diff --
    
    This class is already `private[spark]` so the modifier is redundant.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to