You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@aurora.apache.org by zm...@apache.org on 2017/03/07 04:21:11 UTC

aurora git commit: Remove SerializableClock interface.

Repository: aurora
Updated Branches:
  refs/heads/master 705dbc7cd -> 932e90871


Remove SerializableClock interface.

This removes the `SerializableClock` interface. We are not serializing `Clock`
classes anywhere, so this should be safe to remove.

Reviewed at https://reviews.apache.org/r/57357/


Project: http://git-wip-us.apache.org/repos/asf/aurora/repo
Commit: http://git-wip-us.apache.org/repos/asf/aurora/commit/932e9087
Tree: http://git-wip-us.apache.org/repos/asf/aurora/tree/932e9087
Diff: http://git-wip-us.apache.org/repos/asf/aurora/diff/932e9087

Branch: refs/heads/master
Commit: 932e908713aa3dd3a40c9c57cbabd2bfb8a311c0
Parents: 705dbc7
Author: Zameer Manji <zm...@apache.org>
Authored: Mon Mar 6 20:21:02 2017 -0800
Committer: Zameer Manji <zm...@apache.org>
Committed: Mon Mar 6 20:21:02 2017 -0800

----------------------------------------------------------------------
 .../main/java/org/apache/aurora/common/util/Clock.java   | 11 ++---------
 1 file changed, 2 insertions(+), 9 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/aurora/blob/932e9087/commons/src/main/java/org/apache/aurora/common/util/Clock.java
----------------------------------------------------------------------
diff --git a/commons/src/main/java/org/apache/aurora/common/util/Clock.java b/commons/src/main/java/org/apache/aurora/common/util/Clock.java
index 87afb4e..5c4ced1 100644
--- a/commons/src/main/java/org/apache/aurora/common/util/Clock.java
+++ b/commons/src/main/java/org/apache/aurora/common/util/Clock.java
@@ -13,20 +13,17 @@
  */
 package org.apache.aurora.common.util;
 
-import java.io.Serializable;
-
 /**
  * An abstraction of the system clock.
- *
  * @author John Sirois
  */
 public interface Clock {
-
+  // TODO(zmanji): Consider replacing this with java.time.Clock
   /**
    * A clock that returns the the actual time reported by the system.
    * This clock is guaranteed to be serializable.
    */
-  Clock SYSTEM_CLOCK = new SerializableClock() {
+  Clock SYSTEM_CLOCK = new Clock() {
     @Override public long nowMillis() {
       return System.currentTimeMillis();
     }
@@ -64,7 +61,3 @@ public interface Clock {
   void waitFor(long millis) throws InterruptedException;
 }
 
-/**
- * A typedef to support anonymous {@link Clock} implementations that are also {@link Serializable}.
- */
-interface SerializableClock extends Clock, Serializable { }