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

    https://github.com/apache/spark/pull/21501#discussion_r193494337
  
    --- Diff: 
mllib/src/test/scala/org/apache/spark/ml/feature/StopWordsRemoverSuite.scala ---
    @@ -65,6 +67,56 @@ class StopWordsRemoverSuite extends MLTest with 
DefaultReadWriteTest {
         testStopWordsRemover(remover, dataSet)
       }
     
    +  test("StopWordsRemover with localed input (case insensitive)") {
    +    val stopWords = Array("milk", "cookie")
    +    val remover = new StopWordsRemover()
    +      .setInputCol("raw")
    +      .setOutputCol("filtered")
    +      .setStopWords(stopWords)
    +      .setLocale(new Locale("tr"))
    --- End diff --
    
    Could you leave an inline comment explaining which part is special to tr 
locale?


---

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

Reply via email to