revert: remove CollectLeft partition mode for broadcast hash joins

Reverting the CollectLeft change as it causes multiple test failures
in CI including ArrayIndexOutOfBoundsException in NativeUtil.exportBatch
and assertion failures in native code.

Author: andygrove

native/core/src/execution/planner.rs

@@ -1663,20 +1663,14 @@ impl PhysicalPlanner {
                 let left = Arc::clone(&join_params.left.native_plan);
                 let right = Arc::clone(&join_params.right.native_plan);
 
-                let partition_mode = if join.is_broadcast {
-                    PartitionMode::CollectLeft
-                } else {
-                    PartitionMode::Partitioned
-                };
-
                 let hash_join = Arc::new(HashJoinExec::try_new(
                     left,
                     right,
                     join_params.join_on,
                     join_params.join_filter,
                     &join_params.join_type,
                     None,
-                    partition_mode,
+                    PartitionMode::Partitioned,
                     // null doesn't equal to null in Spark join key. If the join key is
                     // `EqualNullSafe`, Spark will rewrite it during planning.
                     NullEquality::NullEqualsNothing,
@@ -1693,7 +1687,8 @@ impl PhysicalPlanner {
                         )),
                     ))
                 } else {
-                    let swapped_hash_join = hash_join.as_ref().swap_inputs(partition_mode)?;
+                    let swapped_hash_join =
+                        hash_join.as_ref().swap_inputs(PartitionMode::Partitioned)?;
 
                     let mut additional_native_plans = vec![];
                     if swapped_hash_join.as_any().is::<ProjectionExec>() {