Resolve ruff errors for newer version but just ignore them

Author: ntjohnson1

pyproject.toml

@@ -89,7 +89,6 @@ ignore = [
   "FIX002",  # Allow TODO lines - consider removing at some point
   "ISC001",  # Recommended to ignore these rules when using with ruff-format
   "N812",    # Allow importing functions as `F`
-  "PD901",   # Allow variable name df
   "PLR0913", # Allow many arguments in function definition
   "SLF001",  # Allow accessing private members
   "TD002",   # Do not require author names in TODO statements

python/datafusion/input/location.py

@@ -46,7 +46,7 @@ def build_table(
         num_rows = 0  # Total number of rows in the file. Used for statistics
         columns = []
         if file_format == "parquet":
-            import pyarrow.parquet as pq
+            import pyarrow.parquet as pq  # noqa: PLC0415
 
             # Read the Parquet metadata
             metadata = pq.read_metadata(input_item)
@@ -61,7 +61,7 @@ def build_table(
             ]
 
         elif format == "csv":
-            import csv
+            import csv  # noqa: PLC0415
 
             # Consume header row and count number of rows for statistics.
             # TODO: Possibly makes sense to have the eager number of rows

python/datafusion/plan.py

@@ -32,7 +32,7 @@
 ]
 
 
-class LogicalPlan:
+class LogicalPlan:  # noqa: PLW1641
     """Logical Plan.
 
     A `LogicalPlan` is a node in a tree of relational operators (such as

python/tests/test_sql.py

@@ -188,7 +188,7 @@ def test_register_parquet_partitioned(ctx, tmp_path, path_to_str, legacy_data_ty
     partition_data_type = "string" if legacy_data_type else pa.string()
 
     if legacy_data_type:
-        with pytest.warns(DeprecationWarning):
+        with pytest.warns(DeprecationWarning):  # noqa: PT030
             ctx.register_parquet(
                 "datapp",
                 dir_root,