Fix clippy lint for the number of arguments to CsvExec::new()
#11565
Labels
enhancement
New feature or request
CsvExec::new()
#11565
Is your feature request related to a problem or challenge?
As of #11533 ❤️ from @connec the
CsvExec::new()
function gets marked by clippy as having too many argumentsWe have suppressed the clippy error for now, but that is not a good long term solution
Describe the solution you'd like
I think we should fix the clippy lint for real rather than just suppressing it
Describe alternatives you've considered
I would personally suggest a builder pattern such as the following as it is easy to document and extend in the future without breaking API changes
I think we could follow the model of https://docs.rs/datafusion/latest/datafusion/datasource/physical_plan/parquet/struct.ParquetExecBuilder.html
Additional context
No response
The text was updated successfully, but these errors were encountered: