wjones127 commented on code in PR #15184:
URL: https://github.com/apache/arrow/pull/15184#discussion_r1062973706
##########
cpp/src/parquet/metadata.h:
##########
@@ -282,11 +282,20 @@ class PARQUET_EXPORT FileMetaData {
bool Equals(const FileMetaData& other) const;
- /// \brief The number of top-level columns in the schema.
+ /// \brief The number of parquet "leaf" columns.
///
/// Parquet thrift definition requires that nested schema elements are
- /// flattened. This method returns the number of columns in the un-flattened
+ /// flattened. This method returns the number of columns in the flattened
/// version.
+ /// For instance, if the schema looks like this :
+ /// 0 foo.bar
+ /// foo.bar.baz 0
+ /// foo.bar.baz2 1
+ /// foo.qux 2
+ /// 1 foo2 3
+ /// 2 foo3 4
+ /// This method will return 5, because there is 5 "leaf" fields (so 5
Review Comment:
```suggestion
/// This method will return 5, because there are 5 "leaf" fields (so 5
```
--
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]
