mapleFU commented on code in PR #36675:
URL: https://github.com/apache/arrow/pull/36675#discussion_r1263305343
##########
cpp/src/parquet/arrow/reader.h:
##########
@@ -141,26 +141,19 @@ class PARQUET_EXPORT FileReader {
/// \brief Read column as a whole into a chunked array.
///
- /// The indicated column index is relative to the schema
+ /// The index i refers the index of the top level schema field, which may
Review Comment:
I think maybe this is wrong. `ReadColumn` will like:
```
/// foo.bar
/// 0 foo.bar.baz
/// 1 foo.qux
/// 2 foo2
/// 3 foo3
```
It would only works for leaf type. And I think previous `ReadSchemaField`
has bug for 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: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
