interface cleanup

src/daft-plan/src/builder.rs

@@ -27,10 +27,10 @@ impl LogicalPlanBuilder {
 impl LogicalPlanBuilder {
     #[staticmethod]
     pub fn source(filepaths: Vec<String>, schema: &PySchema) -> PyResult<LogicalPlanBuilder> {
-        let source_info = source_info::SourceInfo::FilesInfo(source_info::FilesInfo {
+        let source_info = source_info::SourceInfo::FilesInfo(source_info::FilesInfo::new(
             filepaths,
-            schema: schema.schema.clone(),
-        });
+            schema.schema.clone(),
+        ));
         let logical_plan_builder = LogicalPlanBuilder::from_source(ops::Source::new(
             schema.schema.clone(),
             source_info.into(),

src/daft-plan/src/ops/source.rs

@@ -22,7 +22,7 @@ pub struct Source {
 
 impl Source {
     pub(crate) fn new(schema: SchemaRef, source_info: Arc<SourceInfo>) -> Self {
-        Source {
+        Self {
             schema,
             source_info,
             filters: vec![], // Will be populated by plan optimizer.

src/daft-plan/src/source_info.rs

@@ -17,3 +17,9 @@ pub struct FilesInfo {
     pub filepaths: Vec<String>, // TODO: pull in some sort of URL crate for this
     pub schema: SchemaRef,
 }
+
+impl FilesInfo {
+    pub(crate) fn new(filepaths: Vec<String>, schema: SchemaRef) -> Self {
+        Self { filepaths, schema }
+    }
+}