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

    https://github.com/apache/spark/pull/22048#discussion_r213879659
  
    --- Diff: core/src/test/scala/org/apache/spark/util/UtilsSuite.scala ---
    @@ -1184,6 +1184,25 @@ class UtilsSuite extends SparkFunSuite with 
ResetSystemProperties with Logging {
         
assert(Utils.getSimpleName(classOf[MalformedClassObject.MalformedClass]) ===
           "UtilsSuite$MalformedClassObject$MalformedClass")
       }
    +
    +  test("stringHalfWidth") {
    +    assert(Utils.stringHalfWidth(null) == 0)
    +    assert(Utils.stringHalfWidth("") == 0)
    +    assert(Utils.stringHalfWidth("ab c") == 4)
    +    assert(Utils.stringHalfWidth("1098") == 4)
    +    assert(Utils.stringHalfWidth("mø") == 2)
    +    assert(Utils.stringHalfWidth("γύρ") == 3)
    --- End diff --
    
    Do you mean that the string here needs to be converted to a unicode escape 
form?


---

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

Reply via email to