fix: report error when not literal

GreptimeTeam · Aug 27, 2024 · 5934212 · 5934212
1 parent 498b02e
commit 5934212
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/src/flow/src/expr/func.rs b/src/flow/src/expr/func.rs
@@ -352,6 +352,8 @@ impl UnaryFunc {
                             }.fail()?
                     }
                 };
+
+                // start time argument is optional
                 let start_time = match args.get(2) {
                     Some(start_time) => {
                         if let Some(value) = start_time.expr.as_literal() {
@@ -367,7 +369,10 @@ impl UnaryFunc {
                                 })?;
                             Some(ret)
                         } else {
-                            None
+                            UnexpectedSnafu {
+                                reason: "Expect start time arg to be literal",
+                            }
+                            .fail()?
                         }
                     }
                     None => None,