ngli-me commented on code in PR #6758:
URL: https://github.com/apache/arrow-rs/pull/6758#discussion_r1892147769
##########
arrow-schema/src/schema.rs:
##########
@@ -413,6 +413,81 @@ impl Schema {
&self.metadata
}
+ /// Returns a new schema, normalized based on the max_level
+ /// This carries metadata from the parent schema over as well
+ pub fn normalize(&self, separator: &str, mut max_level: usize) ->
Result<Self, ArrowError> {
Review Comment:
This was to help with the recursion, since using just the helper function
would result in the separator as a prefix in the `field.name()`. I do agree
that this is not the best option though, maybe I can count this as a vote
against the recursion approach? 😆
--
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]
