Minor comments update.

Signed-off-by: Yury-Fridlyand <[email protected]>
opensearch-project · Jan 13, 2023 · 3289103 · 3289103
1 parent 508f1f2
commit 3289103
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 1 deletion.
diff --git a/core/src/main/java/org/opensearch/sql/data/type/ExprCoreType.java b/core/src/main/java/org/opensearch/sql/data/type/ExprCoreType.java
@@ -53,7 +53,6 @@ public enum ExprCoreType implements ExprType {
 
   /**
    * Date.
-   * Todo. compatible relationship.
    */
   DATE(STRING),
   TIME(STRING),

diff --git a/core/src/main/java/org/opensearch/sql/expression/operator/convert/TypeCastOperator.java b/core/src/main/java/org/opensearch/sql/expression/operator/convert/TypeCastOperator.java
@@ -178,6 +178,7 @@ private static DefaultFunctionResolver castToTime() {
     );
   }
 
+  // `DATE`/`TIME`/`DATETIME` -> `DATETIME`/TIMESTAMP` cast tested in BinaryPredicateOperatorTest
   private static DefaultFunctionResolver castToTimestamp() {
     return FunctionDSL.define(BuiltinFunctionName.CAST_TO_TIMESTAMP.getName(),
         impl(nullMissingHandling(
-Original file line number
+Diff line change
@@ Expand Up / @@ -53,7 +53,6 @@ public enum ExprCoreType implements ExprType { @@
       /**
        * Date.
-       * Todo. compatible relationship.
        */
       DATE(STRING),
       TIME(STRING),
@@ Expand Down @@