Minor cleanup

Signed-off-by: currantw <taylor.curran@improving.com>

Author: currantw

docs/ppl-lang/functions/ppl-datetime.md

@@ -402,7 +402,8 @@ Example:
 **Description:**
 
 **Usage:** earliest(string, timestamp) returns whether the timestamp defined by the given relative string is earlier
-than or at the same time as the specified timestamp.
+than or at the same time as the given timestamp. See [RELATIVE_TIMESTAMP](#relative_timestamp)
+for more details on relative timestamp strings.
 
 Argument type: STRING, TIMESTAMP
 
@@ -550,7 +551,8 @@ Example:
 **Description:**
 
 **Usage:** latest(string, timestamp) returns whether the timestamp defined by the given relative string is later
-than or at the same time as the specified timestamp. See [relative_timestamp](#relative_timestamp) for more details.
+than or at the same time as the given timestamp. See [RELATIVE_TIMESTAMP](#relative_timestamp)
+for more details on relative timestamp strings.
 
 Argument type: STRING, TIMESTAMP
 

docs/ppl-lang/ppl-where-command.md

@@ -62,4 +62,4 @@ PPL query:
       | where case(factor = 2, 'even', factor = 4, 'even', factor = 6, 'even', factor = 8, 'even' else 'odd') = 'even'
       |  stats count() by factor`
 - `source = table | where timestamp >= relative_timestamp("-1d@w6")`
-- `source = table | where earliest("-1d@w0", timestamp) and latest("now")`
+- `source = table | where earliest("-1d@w0", timestamp) and latest("now", timestamp)`

integ-test/src/integration/scala/org/opensearch/flint/spark/ppl/FlintSparkPPLBuiltInDateTimeFunctionITSuite.scala

@@ -386,7 +386,7 @@ class FlintSparkPPLBuiltInDateTimeFunctionITSuite
     assertSameRows(Seq(Row(23)), frame)
 
     frame = sql(s"""
-                   | source =$testTable
+                   | source=$testTable
                    | | eval day = day_of_week(relative_timestamp("@w0"))
                    | | fields day
                    | | head 1
@@ -430,32 +430,6 @@ class FlintSparkPPLBuiltInDateTimeFunctionITSuite
     assertSameRows(Seq(Row(false, true, true)), frame)
   }
 
-  // TODO #957: Support earliest
-  ignore("test EARLIEST") {
-    var frame = sql(s"""
-                       | source = $testTable
-                       | | eval earliest_hour_before = earliest(now(), "-1h")
-                       | | eval earliest_now = earliest(now(), "now")
-                       | | eval earliest_hour_after = earliest(now(), "+1h")
-                       | | fields earliest_hour_before, earliest_now, earliest_hour_after
-                       | | head 1
-                       | """.stripMargin)
-    assertSameRows(Seq(Row(true), Row(true), Row(false)), frame)
-  }
-
-  // TODO #957: Support latest
-  ignore("test LATEST") {
-    var frame = sql(s"""
-                       | source = $testTable
-                       | | eval latest_hour_before = latest(now(), "-1h")
-                       | | eval latest_now = latest(now(), "now")
-                       | | eval latest_hour_after = latest(now(), "+1h")
-                       | | fields latest_hour_before, latest_now, latest_hour_after
-                       | | head 1
-                       | """.stripMargin)
-    assertSameRows(Seq(Row(false), Row(true), Row(true)), frame)
-  }
-
   test("test CURRENT_TIME is not supported") {
     val ex = intercept[UnsupportedOperationException](sql(s"""
                        | source = $testTable