pitrou commented on code in PR #242:
URL: https://github.com/apache/parquet-format/pull/242#discussion_r1608989733
##########
src/main/thrift/parquet.thrift:
##########
@@ -467,6 +467,35 @@ struct SchemaElement {
10: optional LogicalType logicalType
}
+struct SchemaElementV3 {
Review Comment:
The problem is that, for any given implementation, we'll have either 1+3
(compiled with old version: backwards compatibility but no perf improvement
when reading), or 2+4 (compiled with new version: better performance on read,
but backwards compatibility is lost).
This doesn't satisfy the requirements we're trying to satisfy here.
--
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]
