Simplify comparators in GcEvent (#1090)

Simplify TIME_ORDER and REVERSE_TIME_ORDER comparators by leveraging Comparator.comparing and .reversed.
Netflix · Oct 31, 2023 · b66ac26 · b66ac26
1 parent 018fd70
commit b66ac26
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/spectator-ext-gc/src/main/java/com/netflix/spectator/gc/GcEvent.java b/spectator-ext-gc/src/main/java/com/netflix/spectator/gc/GcEvent.java
@@ -109,10 +109,8 @@ public String toString() {
   }
 
   /** Order events from oldest to newest. */
-  public static final Comparator<GcEvent> TIME_ORDER =
-      (e1, e2) -> (int) (e1.getStartTime() - e2.getStartTime());
+  public static final Comparator<GcEvent> TIME_ORDER = Comparator.comparing(GcEvent::getStartTime);
 
   /** Order events from newest to oldest. */
-  public static final Comparator<GcEvent> REVERSE_TIME_ORDER =
-      (e1, e2) -> (int) (e2.getStartTime() - e1.getStartTime());
+  public static final Comparator<GcEvent> REVERSE_TIME_ORDER = TIME_ORDER.reversed();
 }