jorisvandenbossche commented on PR #12902: URL: https://github.com/apache/arrow/pull/12902#issuecomment-1102844303
Ah, I missed that you added `date` already to the tests as well. Checking the parquet docs, I think the only other logical type that we don't yet explicitly handled is "interval" (https://github.com/apache/parquet-format/blob/master/LogicalTypes.md#interval), but that is also not yet handled by the C++ `StatisticsAsScalars`, so this won't have changed in behaviour (and in general I don't know the support of mapping our own interval types to parquet interval). @wjones127 would you want to open an issue with kartothek to warn them about the upcoming change? -- 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]
