chore: make MetadataCacheFactory::writer_props return a Result

cube-js · Sep 24, 2024 · c42cad2 · c42cad2
1 parent faf7acb
commit c42cad2
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/datafusion/src/physical_plan/parquet.rs b/datafusion/src/physical_plan/parquet.rs
@@ -153,8 +153,11 @@ 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())
     }
 }