add method SessionStateBuilder::new_with_default_features()

[shruti2522]

Which issue does this PR close?

• Closes Add method to SessionStateBuilder that has all the defaut features #14981

Rationale for this change

What changes are included in this PR?

added method new_with_defaults to StreamStateBuilder

Are these changes tested?

Are there any user-facing changes?

[berkaysynnada]

Why don't we just implement Default trait?

[alamb]

Why don't we just implement Default trait?

There is some more backstory here:

• chore: Update SessionStateBuilder::with_default_features does not replace existing features #14935 (review)

Basically the question is what "defaults" means -- is default an empty builder or does it have all the default functions, etc.

An alternate design might be to always create SessionStateBuilder with all default functions and have a clear() method or something to reset the state.

[alamb]

Thank you @shruti2522 -- this looks great. I think we just need to improve the name and it will be good to go (assuming it is ok with @berkaysynnada and other reviewers)

Could you also add a comment to SessionStateBuilder::new() here:

datafusion/datafusion/core/src/execution/session_state.rs

Lines 1022 to 1025 in b377725

	/// Returns a new empty [`SessionStateBuilder`].
	///
	/// See [`Self::with_default_features`] to install the default set of functions,
	/// catalogs, etc.

Something like "to create a SessionStateBuilder with default features such as functions please see [SessionStateBuilder::new_with_default_features}

[alamb]

To keep the name consistent (and make it harder to confuse this with default()) could we please name this default_features? Like:

Suggested change

	pub fn new_with_defaults() -> Self {
	pub fn new_with_default_features() -> Self {

[shruti2522]

done @alamb