From c42cad29fa47a117958839dda7d03cf572cd86fa Mon Sep 17 00:00:00 2001
From: Sam Hughes <sam@cube.dev>
Date: Mon, 23 Sep 2024 17:51:41 -0700
Subject: [PATCH] chore: make MetadataCacheFactory::writer_props return a
 Result

---
 datafusion/src/physical_plan/parquet.rs | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/datafusion/src/physical_plan/parquet.rs b/datafusion/src/physical_plan/parquet.rs
index da098f1e50b8..5a1fe88d942c 100644
--- 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())
     }
 }