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

Antoine Pitrou commented on PARQUET-1990:
-----------------------------------------

WTH #2, https://github.com/apache/parquet-cpp/pull/360 added a converted type 
named "NA" in order to serialize Arrow null arrays.
https://github.com/apache/parquet-format/pull/45 added a converted type named 
"NA" in the Thrift definition.
https://github.com/apache/parquet-format/pull/51 then removed the converted 
type "NA" (before it was released) and replaced it with the logical type 
"NullType". But the Parquet C++ implementation wasn't updated to also remove 
"NA".

> [C++] ConvertedType::NA is  written out in some cases
> -----------------------------------------------------
>
>                 Key: PARQUET-1990
>                 URL: https://issues.apache.org/jira/browse/PARQUET-1990
>             Project: Parquet
>          Issue Type: Improvement
>          Components: parquet-cpp
>            Reporter: Micah Kornfield
>            Assignee: Micah Kornfield
>            Priority: Major
>
> This makes it an invalid thrift enum.  ::NA is a placeholder enum internal to 
> C++.  There will right out an enum value of 24 to thrift which can cause 
> errors for other readers.  See: 
> https://issues.apache.org/jira/browse/ARROW-10553



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to