[ 
https://issues.apache.org/jira/browse/SPARK-14249?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15257146#comment-15257146
 ] 

Joseph K. Bradley commented on SPARK-14249:
-------------------------------------------

I don't see a good way to do this.  The suggestion of {{PipelineModel.read = 
PipelineModelMLReader(PipelineModel)}} does not actually work since we need 
access to the JVM in the Reader init method, but it is not yet available since 
this is outside the constructor.

I'm going to close this issue.  We'll need to revert the change to write() to 
keep things consistent.  Thanks regardless!

> Change MLReader.read to be a property for PySpark
> -------------------------------------------------
>
>                 Key: SPARK-14249
>                 URL: https://issues.apache.org/jira/browse/SPARK-14249
>             Project: Spark
>          Issue Type: Improvement
>          Components: ML, PySpark
>            Reporter: Joseph K. Bradley
>            Priority: Minor
>
> To match MLWritable.write and SQLContext.read, it will be good to make the 
> PySpark MLReader classmethod {{read}} be a property.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@spark.apache.org
For additional commands, e-mail: issues-h...@spark.apache.org

Reply via email to