etseidl commented on code in PR #8528:
URL: https://github.com/apache/arrow-rs/pull/8528#discussion_r2395292792
##########
parquet/src/basic.rs:
##########
@@ -928,17 +930,31 @@ enum BoundaryOrder {
// ----------------------------------------------------------------------
// Mirrors thrift enum `EdgeInterpolationAlgorithm`
+// TODO(ets): we need to allow for unknown variants. Either hand code this
one, or add a new
+// macro that adds an _Unknown variant.
Review Comment:
Here I'm assuming that an unknown algorithm will result in ignoring the
stats, so should not be fatal.
--
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]
