Convert internal representation of LogicalPlanBuilder from LogicalPlan to Arc<LogicalPlan>

[alamb]

          Great idea @ClSlaid 

Thanks to @AbrarNitk we have a first version of LogicalPlanBuilder::from(arc_input) in #10466 🙏

Now that we have merged that PR we can make a second PR (and maybe a second ticket) about switching the internal representation from

datafusion/datafusion/expr/src/logical_plan/builder.rs

Lines 98 to 101 in 8cc92a9

	#[derive(Debug, Clone)]
	pub struct LogicalPlanBuilder {
	plan: LogicalPlan,
	}

To

 pub struct LogicalPlanBuilder { 
     plan: Arc<LogicalPlan>, 
 } 

The rationale is as explained by @ClSlaid on #10465 (comment) -- that in some cases this can prevent cloning the input

Originally posted by @alamb in #10465 (comment)

[alamb]

I think this is a good first issue as it is pretty clear what the ask is

[iiiancampbell]

take

[alamb]

Awesome -- thank you @iiiancampbell 🎉

[jc4x4]

I noticed the previous commit is stale and closed. Here is my first attempt.
#12040