[
https://issues.apache.org/jira/browse/ARROW-5691?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16914317#comment-16914317
]
Antoine Pitrou commented on ARROW-5691:
---------------------------------------
I also dislike the "adapters" name (too generic, it could be anything).
As for the Parquet-to-Arrow dependency, isn't it broken precisely if the action
outlined in the title is achieved? ("relocate src/parquet/arrow to
src/arrow/dataset/parquet")
I guess we could have a "formats" directory where we would put JSON, CSV,
Parquet, ORC...
Though I'm not fond of deep hierarchies and would also be fine with putting
"json", "csv", "parquet", "orc"... directories at the Arrow top-level.
> [C++] Relocate src/parquet/arrow code to src/arrow/dataset/parquet
> ------------------------------------------------------------------
>
> Key: ARROW-5691
> URL: https://issues.apache.org/jira/browse/ARROW-5691
> Project: Apache Arrow
> Issue Type: Improvement
> Components: C++
> Reporter: Wes McKinney
> Priority: Major
> Fix For: 1.0.0
>
>
> I think it may make sense to continue developing and maintaining this code in
> the same place as other file format <-> Arrow serialization code and dataset
> handling routines (e.g. schema normalization). Under this scheme, libparquet
> becomes a link time dependency of libarrow_dataset
--
This message was sent by Atlassian Jira
(v8.3.2#803003)
