Skip to content

Commit 9d819e1

Browse files
authored
minor: reuse SessionStateBuilder methods for default builder (#12330)
1 parent f638df3 commit 9d819e1

File tree

1 file changed

+9
-11
lines changed

1 file changed

+9
-11
lines changed

datafusion/core/src/execution/session_state.rs

Lines changed: 9 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1035,17 +1035,15 @@ impl SessionStateBuilder {
10351035
}
10361036
}
10371037

1038-
/// Set defaults for table_factories, file formats, expr_planners and builtin
1039-
/// scalar and aggregate functions.
1040-
pub fn with_default_features(mut self) -> Self {
1041-
self.table_factories = Some(SessionStateDefaults::default_table_factories());
1042-
self.file_formats = Some(SessionStateDefaults::default_file_formats());
1043-
self.expr_planners = Some(SessionStateDefaults::default_expr_planners());
1044-
self.scalar_functions = Some(SessionStateDefaults::default_scalar_functions());
1045-
self.aggregate_functions =
1046-
Some(SessionStateDefaults::default_aggregate_functions());
1047-
self.window_functions = Some(SessionStateDefaults::default_window_functions());
1048-
self
1038+
/// Create default builder with defaults for table_factories, file formats, expr_planners and builtin
1039+
/// scalar, aggregate and windows functions.
1040+
pub fn with_default_features(self) -> Self {
1041+
self.with_table_factories(SessionStateDefaults::default_table_factories())
1042+
.with_file_formats(SessionStateDefaults::default_file_formats())
1043+
.with_expr_planners(SessionStateDefaults::default_expr_planners())
1044+
.with_scalar_functions(SessionStateDefaults::default_scalar_functions())
1045+
.with_aggregate_functions(SessionStateDefaults::default_aggregate_functions())
1046+
.with_window_functions(SessionStateDefaults::default_window_functions())
10491047
}
10501048

10511049
/// Set the session id.

0 commit comments

Comments
 (0)