fixup! fixup! fixup! fixup! fixup! fixup! WIP

Author: findepi

datafusion/proto/src/logical_plan/from_proto.rs

@@ -647,7 +647,7 @@ pub fn parse_sort(
 ) -> Result<Sort, Error> {
     Ok(Sort::new(
         Box::new(parse_required_expr(
-            sort.expr.as_deref(),
+            sort.expr.as_ref(),
             registry,
             "expr",
             codec,

datafusion/proto/src/logical_plan/to_proto.rs

@@ -637,7 +637,7 @@ where
                 nulls_first,
             } = sort;
             Ok(protobuf::SortExprNode {
-                expr: Some(Box::new(serialize_expr(expr.as_ref(), codec)?)),
+                expr: Some(serialize_expr(expr.as_ref(), codec)?),
                 asc: *asc,
                 nulls_first: *nulls_first,
             })