chore: fix code comment

GreptimeTeam · May 7, 2024 · a62e33b · a62e33b
1 parent f3001a4
commit a62e33b
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/src/query/src/range_select/plan_rewrite.rs b/src/query/src/range_select/plan_rewrite.rs
@@ -167,10 +167,12 @@ fn evaluate_expr_to_millisecond(args: &[Expr], i: usize) -> DFResult<i64> {
         }
         _ => None,
     }
-    .ok_or(DataFusionError::Plan(format!(
-        "{} is not a expr can be evaluate and use in range query",
-        expr.display_name().unwrap_or_default()
-    )))
+    .ok_or_else(|| {
+        DataFusionError::Plan(format!(
+            "{} is not a expr can be evaluate and use in range query",
+            expr.display_name().unwrap_or_default()
+        ))
+    })
 }
 
 /// Parse the `align to` clause and return a UTC timestamp with unit of millisecond,