modin icon indicating copy to clipboard operation
modin copied to clipboard

Remove or spread *args and **kwargs in BaseIO methods

Open mvashishtha opened this issue 1 year ago • 1 comments

Some BaseIO methods like read_parquet have only **kwargs as their arguments when they should specify particular required arguments. Others, like read_csv, include **kwargs but should not. We should clean up the BaseIO interface so that it only allows parameters that are appropriate for each method.

See also the related issue for the query compiler, #3108.

mvashishtha avatar Mar 19 '24 21:03 mvashishtha

The consequence of this will be a complication of code support (the amount of work to keep signatures up to date will at least double), since function signatures will be defined in full not only at the top level, but also at the level of query compilers.

anmyachev avatar Mar 26 '24 15:03 anmyachev