xhumanoid commented on code in PR #43995:
URL: https://github.com/apache/arrow/pull/43995#discussion_r2601296815
##########
cpp/src/parquet/arrow/arrow_schema_test.cc:
##########
@@ -727,6 +780,60 @@ TEST_F(TestConvertParquetSchema,
ParquetRepeatedNestedSchema) {
ASSERT_NO_FATAL_FAILURE(CheckFlatSchema(arrow_schema));
}
+TEST_F(TestConvertParquetSchema, IllegalParquetNestedSchema) {
Review Comment:
@wgtmac
just to clarify: what to do with old files with legacy encoding? after this
patch we can't read it
now we have exact the same schema, because it's still default for
parquet-java until now
https://github.com/apache/parquet-java/tree/master/parquet-avro#configuration-for-writing
`parquet.avro.write-old-list-structure`
```
Flag whether to write list structures in the old way (2 levels) or the new
one (3 levels).
When writing at 2 levels no null values are available at the element level.
**The default value is true**
```
--
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]
