Compare LocalDateTimes faster by pre-computing epochSecond.

neo4j · Mar 23, 2018 · e04c5db · e04c5db
1 parent 2887607
commit e04c5db
Showing 1 changed file with 10 additions and 3 deletions.
diff --git a/community/values/src/main/java/org/neo4j/values/storable/LocalDateTimeValue.java b/community/values/src/main/java/org/neo4j/values/storable/LocalDateTimeValue.java
@@ -263,17 +263,24 @@ protected LocalDateTimeValue selectDateTime( AnyValue datetime )
     }
 
     private final LocalDateTime value;
+    private final long epochSecondsInUTC;
 
     private LocalDateTimeValue( LocalDateTime value )
     {
         this.value = value;
+        this.epochSecondsInUTC = this.value.toEpochSecond(UTC);
     }
 
     @Override
-    int unsafeCompareTo( Value otherValue )
+    int unsafeCompareTo( Value other )
     {
-        LocalDateTimeValue other = (LocalDateTimeValue) otherValue;
+        LocalDateTimeValue that = (LocalDateTimeValue) other;
-        return value.compareTo( other.value );
+        int cmp = Long.compare( epochSecondsInUTC, that.epochSecondsInUTC );
+        if ( cmp == 0 )
+        {
+            cmp = value.getNano() - that.value.getNano();
+        }
+        return cmp;
     }
 
     @Override