formatting

apache · Sep 23, 2024 · 92b5c58 · 92b5c58
1 parent 78f18f6
commit 92b5c58
Showing 1 changed file with 23 additions and 21 deletions.
diff --git a/native/core/src/execution/datafusion/planner.rs b/native/core/src/execution/datafusion/planner.rs
@@ -1700,9 +1700,9 @@ impl PhysicalPlanner {
                         WindowFrameBound::Preceding(ScalarValue::Int64(None))
                     }
                     WindowFrameUnits::Groups => {
-                        return Err(ExecutionError::GeneralError(format!(
-                            "WindowFrameUnits::Groups is not supported."
-                        )))
+                        return Err(ExecutionError::GeneralError(
+                            "WindowFrameUnits::Groups is not supported.".to_string(),
+                        ));
                     }
                 },
                 LowerFrameBoundStruct::Preceding(offset) => {
@@ -1713,11 +1713,11 @@ impl PhysicalPlanner {
                         )),
                         WindowFrameUnits::Range => {
                             WindowFrameBound::Preceding(ScalarValue::Int64(Some(offset_value)))
-                        },
+                        }
                         WindowFrameUnits::Groups => {
-                            return Err(ExecutionError::GeneralError(format!(
-                                "WindowFrameUnits::Groups is not supported."
-                            )))
+                            return Err(ExecutionError::GeneralError(
+                                "WindowFrameUnits::Groups is not supported.".to_string(),
+                            ));
                         }
                     }
                 }
@@ -1726,10 +1726,11 @@ impl PhysicalPlanner {
             None => match units {
                 WindowFrameUnits::Rows => WindowFrameBound::Preceding(ScalarValue::UInt64(None)),
                 WindowFrameUnits::Range => WindowFrameBound::Preceding(ScalarValue::Int64(None)),
-                WindowFrameUnits::Groups =>
-                    return Err(ExecutionError::GeneralError(format!(
-                        "WindowFrameUnits::Groups is not supported."
-                    )))
+                WindowFrameUnits::Groups => {
+                    return Err(ExecutionError::GeneralError(
+                        "WindowFrameUnits::Groups is not supported.".to_string(),
+                    ));
+                }
             },
         };
 
@@ -1747,9 +1748,9 @@ impl PhysicalPlanner {
                         WindowFrameBound::Following(ScalarValue::Int64(None))
                     }
                     WindowFrameUnits::Groups => {
-                        return Err(ExecutionError::GeneralError(format!(
-                            "WindowFrameUnits::Groups is not supported."
-                        )))
+                        return Err(ExecutionError::GeneralError(
+                            "WindowFrameUnits::Groups is not supported.".to_string(),
+                        ));
                     }
                 },
                 UpperFrameBoundStruct::Following(offset) => match units {
@@ -1760,20 +1761,21 @@ impl PhysicalPlanner {
                         WindowFrameBound::Following(ScalarValue::Int64(Some(offset.offset)))
                     }
                     WindowFrameUnits::Groups => {
-                        return Err(ExecutionError::GeneralError(format!(
-                            "WindowFrameUnits::Groups is not supported."
-                        )))
+                        return Err(ExecutionError::GeneralError(
+                            "WindowFrameUnits::Groups is not supported.".to_string(),
+                        ));
                     }
                 },
                 UpperFrameBoundStruct::CurrentRow(_) => WindowFrameBound::CurrentRow,
             },
             None => match units {
                 WindowFrameUnits::Rows => WindowFrameBound::Following(ScalarValue::UInt64(None)),
                 WindowFrameUnits::Range => WindowFrameBound::Following(ScalarValue::Int64(None)),
-                WindowFrameUnits::Groups =>
-                    return Err(ExecutionError::GeneralError(format!(
-                        "WindowFrameUnits::Groups is not supported."
-                    )))
+                WindowFrameUnits::Groups => {
+                    return Err(ExecutionError::GeneralError(
+                        "WindowFrameUnits::Groups is not supported.".to_string(),
+                    ));
+                }
             },
         };