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] { override def load(path: String): FPGrowthModel = { val metadata = DefaultParamsReader.loadMetadata(path, sc, className) + implicit val format = DefaultFormats + val numTrainingRecords = (metadata.metadata \ "numTrainingRecords").extract[Long] --- End diff -- Yes it does now. If saving numTrainingRecords to FPGrowthModel looks good, I can update the load logic.
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org