Github user sethah commented on the issue:
https://github.com/apache/spark/pull/11119
@MLnick I tend to agree with your opinion about saving the initialModel
with the model.
We can remove saving the initial model entirely from the model. Though,
right now users could avoid saving it by calling
`model.clear(model.initialModel)` before saving. That at least provides
flexibility to do both, but users may not think of it and get tripped up by the
long lineage.
It's probably best to not save it at all in the long run. We can block this
PR until this is resolved or discuss it in a follow up issue - I'm ok with
either. Thanks for bringing this up.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
