[
https://issues.apache.org/jira/browse/PARQUET-1991?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17313046#comment-17313046
]
Gabor Szadovszky commented on PARQUET-1991:
-------------------------------------------
[~apitrou], I agree that we should not add a placeholder to the ConvertedType
enum. Meanwhile, even though we have added the LogicalType union and placed a
couple of instructions about the backward compatibility with the ConvertedTypes
it is not stated clearly that ConvertedType is only there for backward
compatibility and we shall not extend it (in other words it is deprecated). It
is not strictly about this issue so it might worth another jira and PR.
> Reserve ConvertedType==24 due to bug in parquet-cpp implementation
> ------------------------------------------------------------------
>
> Key: PARQUET-1991
> URL: https://issues.apache.org/jira/browse/PARQUET-1991
> Project: Parquet
> Issue Type: Improvement
> Components: parquet-format
> Reporter: Micah Kornfield
> Priority: Major
>
> https://issues.apache.org/jira/browse/PARQUET-1990 sometimes a converted type
> of 24 is written out because we are leaking a C++ enum value that doesn't
> exist in the parquet thrift specification. We should reserve this so if any
> other converted type elements are added future readers of older files don't
> accidentally misinterpret this value.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
