Merge branch '3.7-dev' into 3.8-dev

Author: xiazcy

docs/src/dev/provider/gremlin-semantics.asciidoc

@@ -822,6 +822,7 @@ None
 Incoming date remains unchanged.
 
 *Exceptions*
+
 * If the incoming traverser is a non-String/Number/Date value then an `IllegalArgumentException` will be thrown.
 
 See: link:https://github.com/apache/tinkerpop/tree/x.y.z/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/AsDateStep.java[source],
@@ -851,6 +852,7 @@ list of string instead.
 Null values from the incoming traverser are not processed and remain as null when returned.
 
 *Exceptions*
+
 None
 
 See: link:https://github.com/apache/tinkerpop/tree/x.y.z/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/AsStringGlobalStep.java[source],
@@ -1611,6 +1613,7 @@ The `global` scope will operate on individual string traverser. The `local` scop
 Null values from the incoming traverser are not processed and remain as null when returned.
 
 *Exceptions*
+
 * For `Scope.global` or parameterless function calls, if the incoming traverser is a non-String value then an `IllegalArgumentException` will be thrown.
 * For `Scope.local`, if the incoming traverser is not a string or a list of strings then an `IllegalArgumentException` will be thrown.
 
@@ -1709,6 +1712,7 @@ The `global` scope will operate on individual string traverser. The `local` scop
 Null values from the incoming traverser are not processed and remain as null when returned.
 
 *Exceptions*
+
 * For `Scope.global` or parameterless function calls, if the incoming traverser is a non-String value then an `IllegalArgumentException` will be thrown.
 * For `Scope.local`, if the incoming traverser is not a string or a list of strings then an `IllegalArgumentException` will be thrown.
 
@@ -2097,6 +2101,7 @@ The `global` scope will operate on individual string traverser. The `local` scop
 Null values from the incoming traverser are not processed and remain as null when returned.
 
 *Exceptions*
+
 * For `Scope.global` or parameterless function calls, if the incoming traverser is a non-String value then an `IllegalArgumentException` will be thrown.
 * For `Scope.local`, if the incoming traverser is not a string or a list of strings then an `IllegalArgumentException` will be thrown.
 
@@ -2153,6 +2158,7 @@ The `global` scope will operate on individual string traverser. The `local` scop
 Null values from the incoming traverser are not processed and remain as null when returned.
 
 *Exceptions*
+
 * For `Scope.global` or parameterless function calls, if the incoming traverser is a non-String value then an `IllegalArgumentException` will be thrown.
 * For `Scope.local`, if the incoming traverser is not a string or a list of strings then an `IllegalArgumentException` will be thrown.
 
@@ -2183,6 +2189,7 @@ The `global` scope will operate on individual string traverser. The `local` scop
 Null values from the incoming traverser are not processed and remain as null when returned.
 
 *Exceptions*
+
 * For `Scope.global` or parameterless function calls, if the incoming traverser is a non-String value then an `IllegalArgumentException` will be thrown.
 * For `Scope.local`, if the incoming traverser is not a string or a list of strings then an `IllegalArgumentException` will be thrown.
 
@@ -2215,6 +2222,7 @@ The `global` scope will operate on individual string traverser. The `local` scop
 Null values from the incoming traverser are not processed and remain as null when returned.
 
 *Exceptions*
+
 * For `Scope.global` or parameterless function calls, if the incoming traverser is a non-String value then an `IllegalArgumentException` will be thrown.
 * For `Scope.local`, if the incoming traverser is not a string or a list of strings then an `IllegalArgumentException` will be thrown.
 
@@ -2244,6 +2252,7 @@ The `global` scope will operate on individual string traverser. The `local` scop
 Null values from the incoming traverser are not processed and remain as null when returned.
 
 *Exceptions*
+
 * For `Scope.global` or parameterless function calls, if the incoming traverser is a non-String value then an `IllegalArgumentException` will be thrown.
 * For `Scope.local`, if the incoming traverser is not a string or a list of strings then an `IllegalArgumentException` will be thrown.
 
@@ -2273,6 +2282,7 @@ The `global` scope will operate on individual string traverser. The `local` scop
 Null values from the incoming traverser are not processed and remain as null when returned.
 
 *Exceptions*
+
 * For `Scope.global` or parameterless function calls, if the incoming traverser is a non-String value then an `IllegalArgumentException` will be thrown.
 * For `Scope.local`, if the incoming traverser is not a string or a list of strings then an `IllegalArgumentException` will be thrown.
 
@@ -2301,6 +2311,7 @@ The `global` scope will operate on individual string traverser. The `local` scop
 Null values from the incoming traverser are not processed and remain as null when returned.
 
 *Exceptions*
+
 * For `Scope.global` or parameterless function calls, if the incoming traverser is a non-String value then an `IllegalArgumentException` will be thrown.
 * For `Scope.local`, if the incoming traverser is not a string or a list of strings then an `IllegalArgumentException` will be thrown.
 

gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/DateDiffStep.java

@@ -70,7 +70,7 @@ protected Long map(final Traverser.Admin<S> traverser) {
             } else {
                 // note: null handling consistency to be resolved https://issues.apache.org/jira/browse/TINKERPOP-3152
                 throw new IllegalArgumentException(
-                        String.format("DateDiff can only take OffsetDateTime or Date (deprecated) as argument, encountered %s", object.getClass()));
+                        String.format("DateDiff can only take OffsetDateTime or Date (deprecated) as argument, encountered %s", object == null ? null : object.getClass()));
             }
         } else {
             date = (OffsetDateTime) object;

gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/DateDiffStepTest.java

@@ -136,4 +136,9 @@ public void shouldHandleNullDateWithDateCompatibility() {
 
         assertEquals(now.getTime() / 1000, (long) __.__(now).dateDiff((Date) null).next());
     }
+
+    @Test(expected = IllegalArgumentException.class)
+    public void shouldThrowWhenInputIsNull() {
+        __.__((Object) null).dateDiff(new Date()).next();
+    }
 }