Github user holdenk commented on the pull request:

    https://github.com/apache/spark/pull/11119#issuecomment-181573228
  
    re the first question - I don't think this necessarily needs to be a code 
generated param (although if we do end up having more shared params with 
templated types we should definitely do codegen). For now maybe just a hand 
written HasInitialModel seems fine (although I'd put it in a separate file 
rather than tacking it on the end of the generated code) - but thats just my 
personal thoughts. Maybe @dbtsai can chime in too?


---
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 [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to