GitHub user jkbradley opened a pull request:

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

    [SPARK-5561] [mllib] Generalized PeriodicCheckpointer for RDDs and Graphs

    PeriodicGraphCheckpointer was introduced for Latent Dirichlet Allocation 
(LDA), but it was meant to be generalized to work with Graphs, RDDs, and other 
data structures based on RDDs.  This PR generalizes it.
    
    For those who are not familiar with the periodic checkpointer, it tries to 
automatically handle persisting/unpersisting and checkpointing/removing 
checkpoint files in a lineage of RDD-based objects.
    
    I need it generalized to use with GradientBoostedTrees 
[https://issues.apache.org/jira/browse/SPARK-6684].  It should be useful for 
other iterative algorithms as well.
    
    Changes I made:
    * Copied PeriodicGraphCheckpointer to PeriodicCheckpointer.
    * Within PeriodicCheckpointer, I created abstract methods for the basic 
operations (checkpoint, persist, etc.).
    * The subclasses for Graphs and RDDs implement those abstract methods.
    * I copied the test suite for the graph checkpointer and made tiny 
modifications to make it work for RDDs.
    
    To review this PR, I recommend doing 2 diffs:
    (1) diff between the old PeriodicGraphCheckpointer.scala and the new 
PeriodicCheckpointer.scala
    (2) diff between the 2 test suites
    
    CCing @andrewor14 in case there are relevant changes to checkpointing.
    CCing @feynmanliang in case you're interested in learning about 
checkpointing.
    CCing @mengxr for final OK.
    Thanks all!

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

    $ git pull https://github.com/jkbradley/spark gbt-checkpoint

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

    https://github.com/apache/spark/pull/7728.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 #7728
    
----
commit 568918c99fa9d42f94daecc8d9759794240db148
Author: Joseph K. Bradley <[email protected]>
Date:   2015-07-28T16:48:14Z

    Generalized PeriodicGraphCheckpointer to PeriodicCheckpointer, with 
subclasses for RDDs and Graphs.

----


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