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

Wes McKinney commented on PARQUET-497:
--------------------------------------

See what I did with ColumnReader in PARQUET-485? The file reader does not need 
to know about the physical file structure or how to deserialize the metadata, 
for example. By decoupling these concerns we will be able to test both the 
serialized file layout details and the file reader API independently. 

> Decouple Parquet physical file structure from FileReader class
> --------------------------------------------------------------
>
>                 Key: PARQUET-497
>                 URL: https://issues.apache.org/jira/browse/PARQUET-497
>             Project: Parquet
>          Issue Type: Improvement
>          Components: parquet-cpp
>            Reporter: Wes McKinney
>            Assignee: Wes McKinney
>
> It should be possible to unit test this class without creating an actual 
> Parquet file. We can do this while also keeping the file-based initialization 
> code path (see parquet_reader.cc) about as simple as it is now. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to