[CHORE] Refactor Binary Ops

Cargo.toml

@@ -82,6 +82,8 @@ parquet2 = {path = "src/parquet2"}
 debug = true
 
 [profile.dev]
+debug = "line-tables-only"
+opt-level = 1
 overflow-checks = false
 
 [profile.dev-bench]

src/daft-core/src/array/ops/arithmetic.rs

@@ -59,9 +59,7 @@ where
             let opt_lhs = lhs.get(0);
             match opt_lhs {
                 None => Ok(DataArray::full_null(rhs.name(), lhs.data_type(), rhs.len())),
-                // NOTE: naming logic here is wrong, and assigns the rhs name. However, renaming is handled at the Series level so this
-                // error is obfuscated.
-                Some(lhs) => rhs.apply(|rhs| operation(lhs, rhs)),
+                Some(scalar) => Ok(rhs.apply(|rhs| operation(scalar, rhs))?.rename(lhs.name())),
             }
         }
         (a, b) => Err(DaftError::ValueError(format!(