feat: add validation for parameters in configure_formatter

kosiew · kosiew · commit 23146136e2f8 · 2025-04-29T09:29:03.000+08:00
diff --git a/python/datafusion/html_formatter.py b/python/datafusion/html_formatter.py
@@ -633,6 +633,9 @@ def configure_formatter(**kwargs: Any) -> None:
         **kwargs: Formatter configuration parameters like max_cell_length,
                  max_width, max_height, enable_cell_expansion, etc.
 
+    Raises:
+        ValueError: If any invalid parameters are provided
+
     Example:
         >>> from datafusion.html_formatter import configure_formatter
         >>> configure_formatter(
@@ -642,6 +645,21 @@ def configure_formatter(**kwargs: Any) -> None:
         ...     use_shared_styles=True
         ... )
     """
+    # Valid parameters accepted by DataFrameHtmlFormatter
+    valid_params = {
+        "max_cell_length", "max_width", "max_height", "max_memory_bytes",
+        "min_rows_display", "repr_rows", "enable_cell_expansion", "custom_css",
+        "show_truncation_message", "style_provider", "use_shared_styles"
+    }
+    
+    # Check for invalid parameters
+    invalid_params = set(kwargs) - valid_params
+    if invalid_params:
+        msg = f"Invalid formatter parameters: {', '.join(invalid_params)}. " \
+              f"Valid parameters are: {', '.join(valid_params)}"
+        raise ValueError(msg)
+    
+    # Create and set formatter with validated parameters
     set_formatter(DataFrameHtmlFormatter(**kwargs))
 
 
