[
https://issues.apache.org/jira/browse/PARQUET-481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15148947#comment-15148947
]
Deepak Majeti commented on PARQUET-481:
---------------------------------------
I am adding few tests to check NULLS for the Scanner as part of PARQUET-532.
Making sure we don't duplicate work.
> Refactor and expand reader-test
> -------------------------------
>
> Key: PARQUET-481
> URL: https://issues.apache.org/jira/browse/PARQUET-481
> Project: Parquet
> Issue Type: Improvement
> Components: parquet-cpp
> Affects Versions: cpp-0.1
> Reporter: Aliaksei Sandryhaila
> Assignee: Aliaksei Sandryhaila
> Fix For: cpp-0.1
>
>
> reader-test currently tests with a parquet file and only verifies that we can
> read it, not the correctness of the output.
> Proposed changes:
> - Remove the use of parquet files and use mock objects instead.
> - Move tests for Scanner to scanner-test.cc
> - Get rid of DebugPrint() tests, move to ParquetFilePrinter as a part of
> PARQUET-508.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
