[ 
https://issues.apache.org/jira/browse/SPARK-10023?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yanbo Liang updated SPARK-10023:
--------------------------------
    Description: 
"checkpointInterval" is member of DecisionTreeParams in Scala API which is 
inconsistency with Python API, we should unified them.

* checkpointInterval
** member of DecisionTreeParams <-> Scala API
** shared param used for all ML Transformer/Estimator <-> Python API

Proposal:
"checkpointInterval" is also used at ALS but the meaning for that is different 
from here.
So we make "checkpointInterval" member of DecisionTreeParams for Python API. 
Because it only validate when cacheNodeIds is true and the checkpoint directory 
is set in the SparkContext, it not a common shared param.

  was:
"checkpointInterval" is member of DecisionTreeParams in Scala API which is 
inconsistency with Python API, we should unified them.

* checkpointInterval
** member of DecisionTreeParams <-> Scala API
** shared param used for all ML Transformer/Estimator <-> Python API

Proposal:
 "checkpointInterval" also used at ALS but the meaning for that is different 
from here.
So we make "checkpointInterval" member of DecisionTreeParams for Python API, it 
only validate when cacheNodeIds is true and the checkpoint directory is set in 
the SparkContext.


> Unified DecisionTreeParams "checkpointInterval" between Scala and Python API.
> -----------------------------------------------------------------------------
>
>                 Key: SPARK-10023
>                 URL: https://issues.apache.org/jira/browse/SPARK-10023
>             Project: Spark
>          Issue Type: Sub-task
>          Components: ML, PySpark
>            Reporter: Yanbo Liang
>
> "checkpointInterval" is member of DecisionTreeParams in Scala API which is 
> inconsistency with Python API, we should unified them.
> * checkpointInterval
> ** member of DecisionTreeParams <-> Scala API
> ** shared param used for all ML Transformer/Estimator <-> Python API
> Proposal:
> "checkpointInterval" is also used at ALS but the meaning for that is 
> different from here.
> So we make "checkpointInterval" member of DecisionTreeParams for Python API. 
> Because it only validate when cacheNodeIds is true and the checkpoint 
> directory is set in the SparkContext, it not a common shared param.



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