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

    https://github.com/apache/spark/pull/20366#discussion_r163353940
  
    --- Diff: 
mllib/src/main/scala/org/apache/spark/ml/feature/CountVectorizer.scala ---
    @@ -169,7 +197,7 @@ class CountVectorizer @Since("1.5.0") (@Since("1.5.0") 
override val uid: String)
         }.reduceByKey { case ((wc1, df1), (wc2, df2)) =>
           (wc1 + wc2, df1 + df2)
         }.filter { case (word, (wc, df)) =>
    -      df >= minDf
    +      (df >= minDf) && (df <= maxDf)
    --- End diff --
    
    nit: the parenthesis are not needed


---

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

Reply via email to