Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/17280#discussion_r164942458
--- Diff: mllib/src/main/scala/org/apache/spark/ml/fpm/FPGrowth.scala ---
@@ -319,9 +323,11 @@ object FPGrowthModel extends MLReadable[FPGrowthModel]
{
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/17280#discussion_r161962624
--- Diff: mllib/src/main/scala/org/apache/spark/ml/fpm/FPGrowth.scala ---
@@ -319,9 +323,11 @@ object FPGrowthModel extends MLReadable[FPGrowthModel]
{
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/17280#discussion_r161679593
--- Diff: mllib/src/main/scala/org/apache/spark/ml/fpm/FPGrowth.scala ---
@@ -319,9 +323,11 @@ object FPGrowthModel extends MLReadable[FPGrowthModel]
{
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/17280#discussion_r125736603
--- Diff: python/pyspark/ml/fpm.py ---
@@ -186,29 +186,29 @@ class FPGrowth(JavaEstimator, HasItemsCol,
HasPredictionCol,
|[z] |
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/17280#discussion_r125154331
--- Diff: python/pyspark/ml/fpm.py ---
@@ -186,29 +186,29 @@ class FPGrowth(JavaEstimator, HasItemsCol,
HasPredictionCol,
|[z]
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/17280#discussion_r125154338
--- Diff: python/pyspark/ml/fpm.py ---
@@ -186,29 +186,29 @@ class FPGrowth(JavaEstimator, HasItemsCol,
HasPredictionCol,
|[z]
GitHub user hhbyyh opened a pull request:
https://github.com/apache/spark/pull/17280
[SPARK-19939] [ML] Add support for association rules in ML
## What changes were proposed in this pull request?
jira: https://issues.apache.org/jira/browse/SPARK-19939
Adding another essential