Github user jkbradley commented on the issue:

    https://github.com/apache/spark/pull/16782
  
    > it leaves in place the static class variable for all other ML classes 
that use the wrapper, and those classes continue to use the static class 
variable. 
    
    I think this was discussed above: This WIP PR currently just changes the 
usage for Pipeline, but if the fix is OK for Pipeline, then @BryanCutler can 
update it for all models.
    
    Given the OK from @davies I recommend we proceed with the current fix (but 
using 'self' to hold the kwargs as mentioned above).  With regards to using 
```inspection```, I say we just add a note to the keyword_only wrapper about 
only using it for methods.


---
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