HippoBaro commented on code in PR #9795:
URL: https://github.com/apache/arrow-rs/pull/9795#discussion_r3134159661
##########
parquet/src/file/metadata/mod.rs:
##########
@@ -919,15 +919,10 @@ impl LevelHistogram {
}
}
- /// Updates histogram values using provided repetition levels
- ///
- /// # Panics
- /// if any of the levels is greater than the length of the histogram (
- /// the argument supplied to [`Self::try_new`])
- pub fn update_from_levels(&mut self, levels: &[i16]) {
Review Comment:
Ah that's right. I was under the impression this was one of those
experimental-public symbols.
What do you recommend? Should we keep it? Or formally deprecate it?
--
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]
