Changed check for time

Signed-off-by: Guian Gumpac <[email protected]>
Bit-Quill · Jun 5, 2023 · 3b0c2d2 · 3b0c2d2
1 parent 467d033
commit 3b0c2d2
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/opensearch/src/main/java/org/opensearch/sql/opensearch/data/type/OpenSearchDateType.java b/opensearch/src/main/java/org/opensearch/sql/opensearch/data/type/OpenSearchDateType.java
@@ -229,13 +229,15 @@ private ExprCoreType getExprTypeFromCustomFormatString(String formatString) {
     boolean isTime = false;
 
     try {
+      // Dates will always have "year" in the formatter string
       isDate = formats.stream()
           .anyMatch(format -> DateTimeFormatter.ofPattern(format).toString().toLowerCase().contains("year"));
     } catch (IllegalArgumentException ignore) {
     }
     try {
+      // Times will always have "hour" in the formatter string
       isTime = formats.stream()
-          .anyMatch(format -> DateTimeFormatter.ofPattern(formatString).toString().toLowerCase().contains("minute"));
+          .anyMatch(format -> DateTimeFormatter.ofPattern(format).toString().toLowerCase().contains("hour"));
     } catch (IllegalArgumentException ignore) {
     }