commit 65649348ceceb5630e4c09c8237ab6a2640bd88b
author Sergiy Matusevych <motus@apache.org> Fri Aug 12 14:07:04 2016 -0700
committer Boris Shulman <shulmanb@gmail.com> Fri Aug 26 15:03:50 2016 -0700
tree 0041827fa2e26e118afc9ef85d27ea0f30707f49
parent addd62328d77b5f0f227eb8a448c77dd0a75461e

[REEF-1533] Make RuntimeClock more testable and create unit tests to check graceful and forceful shutdown of RuntimeClock

  * Refactor ClockTest for readability and add more Javadoc comments;
  * Add unit tests for graceful and forceful shutdown of the clock.
  * Move alarm producing functionality into a separate class and factor out the delay logic as an abstract method.
  * Add helper functions to generate random delays to AlarmProducer
  * Move EventRecorder and AlarmProducer into a separate package
  * Refactor the EventRecorder helper class - do not track timestamps separately from events + minor style cleanups.
  * Implement Clock.isClosed() method
  * Minor fixes in other unit tests around the RuntimeClock logic.
  * Log all event subscriptions at the beginning of hte event loop in RuntimeClock
  * Switch to RealTimer instead of LogicalTimer in ClockTest.testAlarmOrder() to avoid race condition (will be addressed in JIRA [REEF-1537](https://issues.apache.org/jira/browse/REEF-1537))

JIRA:
  [REEF-1533](https://issues.apache.org/jira/browse/REEF-1533) close

Pull request:
  Closes [#1099](https://github.com/apache/reef/pull/1099)