arrow icon indicating copy to clipboard operation
arrow copied to clipboard

[C++] shared conversion framework for JSON/CSV parsers

Open asfimport opened this issue 6 years ago • 2 comments

CSV and JSON both convert strings to values in a Array but there is little code sharing beyond arrow::util::StringConverter.

It would be advantageous if a single interface could be shared between CSV and JSON to do the heavy lifting of conversion consistently. This would simplify addition of new parsers as well as allowing all parsers to immediately take advantage of a new conversion strategy.

Reporter: Ben Kietzman / @bkietz

Note: This issue was originally created as ARROW-4706. Please see the migration documentation for further details.

asfimport avatar Feb 27 '19 23:02 asfimport

Antoine Pitrou / @pitrou: I wonder how much can be shared. Obviously the parsing will be different, but even the conversion layer will be different too. So perhaps the higher-level orchestration layer can be shared.

Note that ARROW-3410 will imply changing the CSV orchestration layer.

asfimport avatar Feb 28 '19 09:02 asfimport

This issue has been marked as stale because it has had no activity in the past 365 days. Please remove the stale label or comment below, or this issue will be closed in 14 days. If this improvement is still desired but has no current owner, please add the 'Status: needs champion' label.

github-actions[bot] avatar Dec 14 '25 11:12 github-actions[bot]