Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/22236#discussion_r212833823 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/fpm/AssociationRules.scala --- @@ -107,7 +124,8 @@ object AssociationRules { @Since("1.5.0") val antecedent: Array[Item], @Since("1.5.0") val consequent: Array[Item], freqUnion: Double, - freqAntecedent: Double) extends Serializable { + freqAntecedent: Double, + freqConsequent: Option[Long]) extends Serializable { --- End diff -- Wouldn't we now always know the frequency of the consequent?
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org