pmarks opened a new pull request, #8824: URL: https://github.com/apache/arrow-rs/pull/8824
The old bound was T: ParquetValueType which isn't public, so it was impossible to call the methods on ValueStatistics from a generic function in another crate, forcing the use of macros. Split out a few functions that require T: AsBytes into a separate impl. # Which issue does this PR close? Closes #8823 # Rationale for this change See issue for more details # Are these changes tested? Added a test that call the `ValueStatistics::min_opt` method from a generic function without a bound to a private trait. # Are there any user-facing changes? The methods were already public and documented, they just couldn't actually be called from external crates. -- 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]
