Github user BryanCutler commented on the pull request:

    https://github.com/apache/spark/pull/11906#issuecomment-205537342
  
    > How did you uncover this issue in the first place? The code you used then 
should be usable to create a unit test for this.
    
    I tried to explain in the JIRA, but I probably should have done so here 
also..
    For SPARK-13967, I was adding a binary flag `Param` to the 
`CountVectorizerModel` and when the model is created from the estimator, this 
problem comes up and an exception is raised.
    
    As far as I can tell, this is only a problem when a `Param` is defined in 
the PySpark model only and it is created by an JavaWrapper estimator.  I don't 
see that happening anywhere currently, so I haven't come up with a unit test.  
I could maybe include a temporary version of the `CountVectorizerModel` that I 
was working on, just for testing purposes until I do a PR for SPARK-13967, 
would that be ok?


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