asfimport commented on issue #386: URL: https://github.com/apache/parquet-format/issues/386#issuecomment-2184153985
[Antoine Pitrou](https://issues.apache.org/jira/browse/PARQUET-1991?#comment-17312470) / @pitrou: Several things: 1) this is a Parquet C++ implementation blunder that should not have happened; it doesn't sound very desirable to add a placeholder in the format to guard against misinterpretations of those files 2) converted types are superceded by logical types, so it sounds unlikely that the spec would add new converted types and end up specifying a converted type numbered 24 3) interpretation of said converted type is not necessary, since the NullType logical type should be there as well I would personally recommend closing as won't fix. @gszadovszky Do you have an opinion on this? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
