chore: make MetadataCacheFactory::writer_props return a Result

cube-js · Sep 24, 2024 · 6157bf2 · 6157bf2
1 parent faf7acb
commit 6157bf2
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/datafusion/src/physical_plan/parquet.rs b/datafusion/src/physical_plan/parquet.rs
@@ -153,8 +153,8 @@ pub trait MetadataCacheFactory: Sync + Send {
         time_to_idle: Duration,
     ) -> Arc<dyn ParquetMetadataCache>;
     /// Modifies and builds writer properties.
-    fn build_writer_props(&self, builder: WriterPropertiesBuilder) -> WriterProperties {
-        builder.build()
+    fn build_writer_props(&self, builder: WriterPropertiesBuilder) -> Result<WriterProperties> {
+        Ok(builder.build())
     }
 }