GitHub user sethah opened a pull request:

    https://github.com/apache/spark/pull/12097

    [SPARK-14308][ML][MLLIB] Remove unused mllib tree classes and move private 
classes to ML

    ## What changes were proposed in this pull request?
    
    Decision tree helper classes will be migrated to ML. This patch moves those 
internal classes that are not part of the public API and removes ones that are 
no longer used, after 
[SPARK-12183](https://github.com/apache/spark/pull/11855). No functional 
changes are made.
    
    Details:
    * Bin.scala is removed as the ML implementation does not require bins
    * mllib NodeIdCache is removed. It was only used by the mllib 
implementation previously, which no longer exists
    * mllib TreePoint is removed. It was only used by the mllib implementation 
previously, which no longer exists
    * BaggedPoint, DTStatsAggregator, DecisionTreeMetadata, BaggedPointSuite 
and TimeTracker are all moved to ML. 
    
    ## How was this patch tested?
    
    No functional changes are made. Existing unit tests ensure behavior is 
unchanged.


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/sethah/spark cleanup_mllib_tree

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/12097.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #12097
    
----
commit 186262fd2eb59ab0ff32bfc986de3b952b9bc917
Author: sethah <[email protected]>
Date:   2016-03-31T18:41:50Z

    cleanup mllib impl

commit ec95de17627daf98d5ba53821b7e1bf4f1e67689
Author: sethah <[email protected]>
Date:   2016-03-31T18:54:18Z

    moving bagged point test suite

commit 3fb50f27ea41b18461fd4e507c1af375622c7236
Author: sethah <[email protected]>
Date:   2016-03-31T20:23:37Z

    moving metadata and stats aggregator

commit 7ec154326d8f82f350464f571b1b5dd3798b2b67
Author: sethah <[email protected]>
Date:   2016-03-31T21:07:44Z

    style fixes

----


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