[
https://issues.apache.org/jira/browse/PARQUET-1990?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17312462#comment-17312462
]
Micah Kornfield commented on PARQUET-1990:
------------------------------------------
Nice find on the reverted format change. I added a DCHECK to ensure only
value less then NA were written out and all tests still pass, I can open a PR
with that change, or if you were thinking of ripping out NA entirely I can hold
off.
> [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: Antoine Pitrou
> Priority: Major
> Fix For: cpp-4.0.0
>
>
> 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)
