Addressing PR 2 comments.

Signed-off-by: Mitchell Gale <[email protected]>

Author: MitchellGale

build.gradle

@@ -87,7 +87,7 @@ spotless {
             include 'core/src/main/java/org/opensearch/sql/monitor/**/*.java',
                     'core/src/main/java/org/opensearch/sql/expression/**/*.java',
                     'core/src/main/java/org/opensearch/sql/executor/**/*.java',
-                    'core/src/main/java/org/opensearch/sql/exception/**/*.java'
+                    'core/src/main/java/org/opensearch/sql/exception/**/*.java',
                     'core/src/main/java/org/opensearch/sql/analysis/**/*.java',
                     'core/src/test/java/org/opensearch/sql/data/**/*.java',
                     'core/src/test/java/org/opensearch/sql/datasource/**/*.java',

core/src/main/java/org/opensearch/sql/executor/pagination/CanPaginateVisitor.java

@@ -52,12 +52,12 @@
  * Currently, V2 engine does not support queries with:<br>
  *
  * <ul>
- *   <li>aggregation (GROUP BY clause or aggregation functions like min/max)
- *   <li>in memory aggregation (window function)
- *   <li>LIMIT/OFFSET clause(s)
- *   <li>without FROM clause
- *   <li>JOIN
- *   <li>a subquery
+ *   <li>aggregation (GROUP BY clause or aggregation functions like min/max)</li>
+ *   <li>in memory aggregation (window function)</li>
+ *   <li>LIMIT/OFFSET clause(s)</li>
+ *   <li>without FROM clause</li>
+ *   <li>JOIN</li>
+ *   <li>a subquery</li>
  * </ul>
  *
  * V2 also requires that the table being queried should be an OpenSearch index.<br>

core/src/main/java/org/opensearch/sql/expression/datetime/DateTimeFunction.java

@@ -1716,10 +1716,10 @@ private ExprValue exprLastDayToday(Clock clock) {
    * Furthermore:<br>
    *
    * <ul>
-   *   <li>zero year interpreted as 2000
-   *   <li>negative year is not accepted
-   *   <li>@dayOfYear should be greater than 1
-   *   <li>if @dayOfYear is greater than 365/366, calculation goes to the next year(s)
+   *   <li>zero year interpreted as 2000</li>
+   *   <li>negative year is not accepted</li>
+   *   <li>@dayOfYear should be greater than 1</li>
+   *   <li>if @dayOfYear is greater than 365/366, calculation goes to the next year(s)</li>
    * </ul>
    *
    * @param yearExpr year

core/src/main/java/org/opensearch/sql/expression/operator/arthmetic/MathematicalFunction.java

@@ -114,7 +114,7 @@ private static DefaultFunctionResolver baseMathFunction(
   /**
    * <b>Definition of abs() function.<\b><br>
    * The supported signature of abs() function are<br>
-   * INT -> INT LONG -> LONG FLOAT -> FLOAT DOUBLE -> DOUBLE
+   * INT/LONG/FLOAT/DOUBLE -> INT/LONG/FLOAT/DOUBLE
    */
   private static DefaultFunctionResolver abs() {
     return define(