Always add parentheses when formatting BinaryExpr with SchemaDisplay

datafusion/expr/src/expr.rs

@@ -2500,7 +2500,7 @@ impl Display for SchemaDisplay<'_> {
                 }
             }
             Expr::BinaryExpr(BinaryExpr { left, op, right }) => {
-                write!(f, "{} {op} {}", SchemaDisplay(left), SchemaDisplay(right),)
+                write!(f, "({} {op} {})", SchemaDisplay(left), SchemaDisplay(right),)
             }
             Expr::Case(Case {
                 expr,
@@ -3577,6 +3577,21 @@ mod test {
         );
     }
 
+    #[test]
+    fn test_schema_display_nested_binary_expr() {
+        let expr = lit(1) * (lit(2) + lit(3));
+        assert_eq!(
+            format!("{}", SchemaDisplay(&expr)),
+            "(Int32(1) * (Int32(2) + Int32(3)))"
+        );
+
+        let expr = -(lit(1) + (lit(2)));
+        assert_eq!(
+            format!("{}", SchemaDisplay(&expr)),
+            "(- (Int32(1) + (Int32(2))"
+        );
+    }
+
     fn wildcard_options(
         opt_ilike: Option<IlikeSelectItem>,
         opt_exclude: Option<ExcludeSelectItem>,