GitHub user andrewor14 opened a pull request:

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

    Add tests for FileLogger, EventLoggingListener, and ReplayListenerBus

    Modifications to Spark core are limited to exposing functionality to test 
files + minor style fixes.

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

    $ git pull https://github.com/andrewor14/spark event-log-tests

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

    https://github.com/apache/spark/pull/591.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 #591
    
----
commit e12f4b1ef342b6f0a1b2188904951bb4256263b2
Author: Andrew Or <[email protected]>
Date:   2014-04-11T21:16:49Z

    Preliminary tests for EventLoggingListener (need major cleanup)

commit 5d38ffe98d9b15ce88518cdb50420fe6c39a04e2
Author: Andrew Or <[email protected]>
Date:   2014-04-11T21:43:49Z

    Clean up EventLoggingListenerSuite + add comments

commit 769336f33e7800c4dc158e3926d74d4480597629
Author: Andrew Or <[email protected]>
Date:   2014-04-28T17:21:09Z

    Merge branch 'master' of github.com:apache/spark into event-log-tests

commit 187bb2587b6df07a77a72d4be19e1ed0d12489ff
Author: Andrew Or <[email protected]>
Date:   2014-04-28T17:34:37Z

    Formatting and renaming variables

commit ab66a846b23bcc16377afd35ec9b897f921b4626
Author: Andrew Or <[email protected]>
Date:   2014-04-29T03:11:47Z

    Tests for FileLogger + delete file after tests

commit b9904537ca19e3e18504a1f448cc77958bbbb909
Author: Andrew Or <[email protected]>
Date:   2014-04-29T05:16:41Z

    ReplayListenerBus suite - tests do not all pass yet

commit e0ba2f8e8c37adc4c57684b2a42aa20fc2e19b19
Author: Andrew Or <[email protected]>
Date:   2014-04-29T20:25:04Z

    Fix test failures caused by race condition in processing/mutating events
    
    An event can be mutated by the DAGScheduler in between being procssed by one
    listener and being processed by another. This causes the ReplayListenerSuite
    to be flaky. This commit ensures that the event logged is the same as the
    original event received by the EventLoggingListener.

commit 862e752d7500d7f89c0a005123507fecb8911fc4
Author: Andrew Or <[email protected]>
Date:   2014-04-29T20:32:08Z

    Merge branch 'master' of github.com:apache/spark into event-log-tests
    
    Conflicts:
        
core/src/main/scala/org/apache/spark/scheduler/EventLoggingListener.scala
        core/src/main/scala/org/apache/spark/util/FileLogger.scala

commit ad2beff5226b4ff93d36252cfceca883cfb789fa
Author: Andrew Or <[email protected]>
Date:   2014-04-29T21:17:45Z

    Clean up EventLoggingListenerSuite + modify a few comments

commit 62010fd3a5f9ca08eedcf6fb41f37d6094b89295
Author: Andrew Or <[email protected]>
Date:   2014-04-29T21:25:11Z

    More cleanup (renaming variables, updating comments etc)

----


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

Reply via email to