[ 
https://issues.apache.org/jira/browse/ARROW-5691?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16913863#comment-16913863
 ] 

Micah Kornfield commented on ARROW-5691:
----------------------------------------

Given the current organization of the code base and based on [~xhochy] comment 
above.  I think we should put the core logic of reading files under the adaptor 
folders (where ORC is currently located), then consume that from datasets.  I 
don't have a good mental model of the current .so dependencies to offer a 
meaningful opinion on that. 

 

 

> [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)

Reply via email to