Skip to content

REF: consolidate in CSV parser module #39345

Open
@arw2019

Description

@arw2019

Follow-up to #38930, mentioned in #39217 and #38370

We'd like to consolidate the parser classes (C, python and pyarrow). One way to go would be to move the logic common to all parsers into ParserBase and delegate parser-specific logic into methods on the subclasses that ParserBase calls

IMO this is best done after the initial pyarrow engine PR (#38370) is merged (so as to to kill all the birds with one stone) but ofc completely up to whoever is doing the work. Creating this issue to track the discussion(s) in either case

cc @jreback @phofl

Metadata

Metadata

Assignees

No one assigned

    Labels

    IO CSVread_csv, to_csvRefactorInternal refactoring of code

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions