emkornfield commented on code in PR #50909:
URL: https://github.com/apache/arrow/pull/50909#discussion_r4107159893
##########
cpp/src/parquet/schema.cc:
##########
@@ -50,6 +50,29 @@ void CheckColumnBounds(int column_index, size_t max_columns)
{
}
}
+std::string ColumnPathFromParquet(const SchemaElement* schema,
Review Comment:
it seems that this is needed because the existing methods assume unflatten
has already happened. I think in might be possible to thread the tree walk
through unflatten and reuse `ToDotString` to avoid the added method here.
Thoughts?
--
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]
